mxc/gpio: make _set_value work with values != 0/1
authorPeter Korsgaard <jacmet@sunsite.dk>
Mon, 11 Oct 2010 13:02:06 +0000 (15:02 +0200)
committerSascha Hauer <s.hauer@pengutronix.de>
Mon, 11 Oct 2010 13:31:40 +0000 (15:31 +0200)
Documentation/gpio.txt specifies that the value argument to
gpio_set_value() should be handled as a boolean (E.G. != 0 is high),
so use the same logic as in _set_direction().

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
arch/arm/plat-mxc/gpio.c

index 57ec4a8..9d38da0 100644 (file)
@@ -235,7 +235,7 @@ static void mxc_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
        unsigned long flags;
 
        spin_lock_irqsave(&port->lock, flags);
-       l = (__raw_readl(reg) & (~(1 << offset))) | (value << offset);
+       l = (__raw_readl(reg) & (~(1 << offset))) | (!!value << offset);
        __raw_writel(l, reg);
        spin_unlock_irqrestore(&port->lock, flags);
 }