________________________________________ > From: Kevin Hilman [[email protected]] > Sent: Wednesday, April 07, 2010 4:21 AM > To: Varadarajan, Charulatha > Cc: [email protected] > Subject: Re: [PATCH] OMAP: GPIO: Fix OMAP4 GPIO reg access issues > > Charulatha V <[email protected]> writes: > > > Access to some of the OMAP4 GPIO registers are not properly handled. > > This patch fixes it. > > > > Tested on... ?
Tested on 3430SDP and 4430SDP. > > Signed-off-by: Charulatha V <[email protected]> > --- [...] > > @@ -1570,9 +1603,14 @@ static int gpio_is_input(struct gpio_bank *bank, int > > mask) > > reg += OMAP7XX_GPIO_DIR_CONTROL; > > break; > > case METHOD_GPIO_24XX: > > - case METHOD_GPIO_44XX: > > reg += OMAP24XX_GPIO_OE; > > break; > > + case METHOD_GPIO_44XX: > > + reg += OMAP4_GPIO_OE; > > + break; > > + default: > > + WARN_ON(1); > > + return -EINVAL; > > If you add a WARN, please add a helpful string. Added this WARN in the same way as in other functions of the same file. > > Otherwise, this looks OK to me. > > Acked-by: Kevin Hilman <[email protected]> Thanks.-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
