From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?iso-8859-1?q?St=FCbner?= Subject: Re: [PATCH] i2c: s3c2410: check for NULL pinctrl handle Date: Mon, 25 Feb 2013 00:16:49 +0100 Message-ID: <201302250016.49773.heiko@sntech.de> References: <201302231855.47560.heiko@sntech.de> <1366216.NzXQfD1BG7@flatron> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org To: Linus Walleij Cc: Tomasz Figa , Wolfram Sang , linux-kernel@vger.kernel.org, Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-i2c@vger.kernel.org, Tomasz Figa List-Id: linux-i2c@vger.kernel.org Am Sonntag, 24. Februar 2013, 23:39:44 schrieb Linus Walleij: > On Sun, Feb 24, 2013 at 6:00 PM, Tomasz Figa wrote: > >> > Note that we are talking here about a temporary solution. The legacy > >> > DT- based pin configuration will go away after all the DT-enabled > >> > platforms using this driver get migrated to pin control and so will > >> > the need to check if pin control is available. > >> > >> So use AUXDATA, and you get a pdata for that driver? > > > > Hmm, and then have some platform data passed statically and some parsed > > from device tree? > > This is done by several in-tree drivers today. It is even necessary for > things like machine-specific callbacks. > > > Not even saying that we are going towards getting rid of > > auxdata, not adding further dependencies for it. > > The other option is to do the non-temporary solution you are > referring to below... > > > Sorry, but this sounds more broken to me than checking the return value > > of devm_pinctrl_get_select_default for NULL in the driver. > > Both are bad solution, auxdata is less bad than trying to check > struct pinctrl * handles for non-NULL, which has *never* been a > good thing to do and should never have been merged in the first > place. > > (Maybe I ACKed that, then I was doing something stupid.) > > > Still, all the platforms relying on the legacy DT GPIO support should > > have been already migrated to pin control, so ideally instead of > > "fixing" the drivers to continue supporting the deprecated method, such > > platforms should be fixed. > > I agree. Fine by me ... I'll work on a pinctrl driver for s3c24xx then :-) Heiko From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko =?iso-8859-1?q?St=FCbner?=) Date: Mon, 25 Feb 2013 00:16:49 +0100 Subject: [PATCH] i2c: s3c2410: check for NULL pinctrl handle In-Reply-To: References: <201302231855.47560.heiko@sntech.de> <1366216.NzXQfD1BG7@flatron> Message-ID: <201302250016.49773.heiko@sntech.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am Sonntag, 24. Februar 2013, 23:39:44 schrieb Linus Walleij: > On Sun, Feb 24, 2013 at 6:00 PM, Tomasz Figa wrote: > >> > Note that we are talking here about a temporary solution. The legacy > >> > DT- based pin configuration will go away after all the DT-enabled > >> > platforms using this driver get migrated to pin control and so will > >> > the need to check if pin control is available. > >> > >> So use AUXDATA, and you get a pdata for that driver? > > > > Hmm, and then have some platform data passed statically and some parsed > > from device tree? > > This is done by several in-tree drivers today. It is even necessary for > things like machine-specific callbacks. > > > Not even saying that we are going towards getting rid of > > auxdata, not adding further dependencies for it. > > The other option is to do the non-temporary solution you are > referring to below... > > > Sorry, but this sounds more broken to me than checking the return value > > of devm_pinctrl_get_select_default for NULL in the driver. > > Both are bad solution, auxdata is less bad than trying to check > struct pinctrl * handles for non-NULL, which has *never* been a > good thing to do and should never have been merged in the first > place. > > (Maybe I ACKed that, then I was doing something stupid.) > > > Still, all the platforms relying on the legacy DT GPIO support should > > have been already migrated to pin control, so ideally instead of > > "fixing" the drivers to continue supporting the deprecated method, such > > platforms should be fixed. > > I agree. Fine by me ... I'll work on a pinctrl driver for s3c24xx then :-) Heiko