power: gpio-charger: do not use gpio value directly

Some gpio implementations return interesting values for gpio_get_value when
the value is not 0 - as seen on a imx6sl board. Therefore do not use the
value returned from gpio_get_value directly but simply check for 0 or not 0.

Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>
Reviewed-by: Doug Anderson <dianders@chromium.org>
Tested-by: Doug Anderson <dianders@chromium.org>
Signed-off-by: Sebastian Reichel <sre@kernel.org>

diff --git a/drivers/power/gpio-charger.c b/drivers/power/gpio-charger.c
index a0024b252197d195a8a1170b68d2c28ad8f3696b..7536933d0ab977574ec3138c47f5c5028028a084 100644 (file)
--- a/drivers/power/gpio-charger.c
+++ b/drivers/power/gpio-charger.c
@@ -55,7 +55,7 @@ static int gpio_charger_get_property(struct power_supply *psy,
 
        switch (psp) {
        case POWER_SUPPLY_PROP_ONLINE:
-               val->intval = gpio_get_value_cansleep(pdata->gpio);
+               val->intval = !!gpio_get_value_cansleep(pdata->gpio);
                val->intval ^= pdata->gpio_active_low;
                break;
        default: