From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933471Ab3FRRTi (ORCPT ); Tue, 18 Jun 2013 13:19:38 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:40324 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933106Ab3FRRTh convert rfc822-to-8bit (ORCPT ); Tue, 18 Jun 2013 13:19:37 -0400 From: "J, KEERTHY" To: Stephen Warren CC: "linux-omap@vger.kernel.org" , "broonie@kernel.org" , "ldewangan@nvidia.com" , "sameo@linux.intel.com" , "grant.likely@secretlab.ca" , "swarren@nvidia.com" , "linux-kernel@vger.kernel.org" , "linux-doc@vger.kernel.org" , "gg@slimlogic.co.uk" Subject: RE: [PATCH v2 1/4] MFD: Palmas: Check if interrupts property exists and then only request irq Thread-Topic: [PATCH v2 1/4] MFD: Palmas: Check if interrupts property exists and then only request irq Thread-Index: AQHObAxY8SVidqm7fEWvMR88AcSmApk7GTuAgACV5XD//3+CgIAAhmbw Date: Tue, 18 Jun 2013 17:19:14 +0000 Message-ID: References: <1371549692-7361-1-git-send-email-j-keerthy@ti.com> <1371549692-7361-2-git-send-email-j-keerthy@ti.com> <51C0820B.4020404@wwwdotorg.org> <51C093FF.6000204@wwwdotorg.org> In-Reply-To: <51C093FF.6000204@wwwdotorg.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.24.170.142] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Stephen Warren [mailto:swarren@wwwdotorg.org] > Sent: Tuesday, June 18, 2013 10:38 PM > To: J, KEERTHY > Cc: linux-omap@vger.kernel.org; broonie@kernel.org; > ldewangan@nvidia.com; sameo@linux.intel.com; grant.likely@secretlab.ca; > swarren@nvidia.com; linux-kernel@vger.kernel.org; linux- > doc@vger.kernel.org; gg@slimlogic.co.uk > Subject: Re: [PATCH v2 1/4] MFD: Palmas: Check if interrupts property > exists and then only request irq > > On 06/18/2013 10:54 AM, J, KEERTHY wrote: > > Hi Stephen, > > > >> -----Original Message----- > >> From: Stephen Warren [mailto:swarren@wwwdotorg.org] > >> Sent: Tuesday, June 18, 2013 9:22 PM > >> To: J, KEERTHY > >> Cc: linux-omap@vger.kernel.org; broonie@kernel.org; > >> ldewangan@nvidia.com; sameo@linux.intel.com; > >> grant.likely@secretlab.ca; swarren@nvidia.com; > >> linux-kernel@vger.kernel.org; linux- doc@vger.kernel.org; > >> gg@slimlogic.co.uk > >> Subject: Re: [PATCH v2 1/4] MFD: Palmas: Check if interrupts > property > >> exists and then only request irq > >> > >> On 06/18/2013 04:01 AM, J Keerthy wrote: > >>> Check if interrupts property exists and then only request irq. > >>> On some boards INT line might not be connected to a valid irq line > >>> on the application processor. Hence keeping a check before > >>> requesting irq. > >> > >> When there is no interrupts property, surely i2c->irq == 0, which is > >> an invalid IRQ, and hence there's no need to check this before > >> copying the value? > > > > The intent here is NOT to request irq with 0 or Invalid IRQ. > > Sure. > > > The board File will not populate the interrupts entry if the INT line > > is not Connected. > > Do you mean the interrupts DT property won't be present if there is no > interrupt. If so, sure. Yes. > > > Hence the patch checks for the 'interrupts' property. > > That shouldn't be necessary; IIRC, the I2C core has already parsed the > interrupts property if there was one, and if there wasn't, it has set > i2c->irq to some invalid value already. > > So, you simply need to check the value in i2c->irq, and don't need to > look at the DT at all. Instead of checking the Invalid irq value which most likely can be 0. I am not sure. I am explicitly checking if the interrupts property exists or not. If not present then It throws out a warning. Either there is no Valid INT line connection or the DeviceTree was not populated fully. This additional piece of information is good to have in the driver IMHO. Let me know if this is rational enough to have in the driver. Regards, Keerthy