diff options
author | Linus Walleij <linus.walleij@linaro.org> | 2014-02-05 08:08:02 -0500 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2014-02-06 04:33:48 -0500 |
commit | 25b35da7f4cce82271859f1b6eabd9f3bd41a2bb (patch) | |
tree | 4ac95fc298a149ec24c8517bbf04db27d1c9aa6e /drivers/gpio | |
parent | f86b7c70bc03236188c9a865aacd57b8d1ddb08c (diff) |
gpio: generic: clamp retured value to [0,1]
The generic GPIO would return 0 for low generic GPIO, and
something != 0 for high GPIO. Let's make this sane by clamping
the returned value to [0,1].
Reported-by: Evgeny Boger <boger@contactless.ru>
Reviewed-by: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio')
-rw-r--r-- | drivers/gpio/gpio-generic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c index 8c778afdf49f..d815dd25eb47 100644 --- a/drivers/gpio/gpio-generic.c +++ b/drivers/gpio/gpio-generic.c | |||
@@ -139,7 +139,7 @@ static int bgpio_get(struct gpio_chip *gc, unsigned int gpio) | |||
139 | { | 139 | { |
140 | struct bgpio_chip *bgc = to_bgpio_chip(gc); | 140 | struct bgpio_chip *bgc = to_bgpio_chip(gc); |
141 | 141 | ||
142 | return bgc->read_reg(bgc->reg_dat) & bgc->pin2mask(bgc, gpio); | 142 | return !!(bgc->read_reg(bgc->reg_dat) & bgc->pin2mask(bgc, gpio)); |
143 | } | 143 | } |
144 | 144 | ||
145 | static void bgpio_set(struct gpio_chip *gc, unsigned int gpio, int val) | 145 | static void bgpio_set(struct gpio_chip *gc, unsigned int gpio, int val) |