gpio: pxa: clamp gpio get value to [0,1]
authorNeil Zhang <zhangwm@marvell.com>
Thu, 9 Jan 2014 09:25:57 +0000 (17:25 +0800)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 15 Jan 2014 07:56:20 +0000 (08:56 +0100)
It would be convenient to normalize the return value for gpio_get.

I have checked mach-mmp / mach-pxa / plat-pxa / plat-orion / mach-orion5x.
It's OK for all of them to change this function to return 0 and 1.

Signed-off-by: Neil Zhang <zhangwm@marvell.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-pxa.c

index cc13d1b74fad09f06c0d2bd42cc4d41b196df2cf..42e6e64f2120364857d551df2a0de579ce7bec2d 100644 (file)
@@ -263,7 +263,8 @@ static int pxa_gpio_direction_output(struct gpio_chip *chip,
 
 static int pxa_gpio_get(struct gpio_chip *chip, unsigned offset)
 {
-       return readl_relaxed(gpio_chip_base(chip) + GPLR_OFFSET) & (1 << offset);
+       u32 gplr = readl_relaxed(gpio_chip_base(chip) + GPLR_OFFSET);
+       return !!(gplr & (1 << offset));
 }
 
 static void pxa_gpio_set(struct gpio_chip *chip, unsigned offset, int value)