From mboxrd@z Thu Jan 1 00:00:00 1970 From: balbi@ti.com (Felipe Balbi) Date: Tue, 5 Apr 2011 15:03:39 +0300 Subject: [PATCHv2] omap: rx51: Platform support for lp5523 led chip In-Reply-To: <4D9B044B.7000604@nokia.com> References: <1301578692-29286-1-git-send-email-ameya.palande@nokia.com> <20110331142635.GF5465@legolas.emea.dhcp.ti.com> <4D949E18.6090704@nokia.com> <20110331154421.GN5465@legolas.emea.dhcp.ti.com> <4D9B044B.7000604@nokia.com> Message-ID: <20110405120339.GD30505@legolas.emea.dhcp.ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Apr 05, 2011 at 03:00:11PM +0300, Ameya Palande wrote: > Hi Felipe, > > On 03/31/2011 06:44 PM, ext Felipe Balbi wrote: > >On Thu, Mar 31, 2011 at 06:30:32PM +0300, Ameya Palande wrote: > >>Hi Felipe, > >> > >>On 03/31/2011 05:26 PM, ext Felipe Balbi wrote: > >>>Hi, > >>> > >>>On Thu, Mar 31, 2011 at 04:38:12PM +0300, Ameya Palande wrote: > >>>>+static int rx51_lp5523_setup(void) > >>>>+{ > >>>>+ int err; > >>>>+ > >>>>+ err = gpio_request_one(RX51_LP5523_CHIP_EN_GPIO, GPIOF_DIR_OUT, > >>>>+ "lp5523_enable"); > >>>>+ if (err< 0) { > >>>>+ pr_err("Unable to get lp5523_enable GPIO\n"); > >>>>+ return err; > >>>>+ } > >>>>+ > >>>>+ return err; > >>> > >>>isn't enough to return gpio_request_one(RX51_LP5523_CHIP_EN_GPIO, > >>>GPIOF_DIR_OUT, "lp5523_enable"); ?? > >> > >>But then we won't know the failure reason for lp5523_probe() > >>I would prefer printing an error! > > > >see that both gpio_request() and gpio_direction_output() have debugging > >prints for error cases ;-) > > Any comments about the version 3 of this patch? looks good to me. Replied with my Reviewed-by tag. -- balbi