diff options
| author | Andreas Kemnade <andreas@kemnade.info> | 2020-12-04 15:49:50 +0100 |
|---|---|---|
| committer | Sebastian Reichel <sre@kernel.org> | 2021-01-16 15:44:58 +0100 |
| commit | cd060b4d0868c806c2738a5e64e8ab9bd0fbec07 (patch) | |
| tree | ee3b5c027526f87febe95b1af8da0b02f28023cd /drivers/power/supply/bq27xxx_battery.c | |
| parent | 95b78d53706d3866c232e4f4819ad38a88f148d4 (diff) | |
| download | linux-cd060b4d0868c806c2738a5e64e8ab9bd0fbec07.tar.bz2 | |
power: supply: bq27xxx: fix polarity of current_now
current_now has to be negative during discharging and positive during
charging, the behavior seen is the other way round.
Tested on GTA04 with Openmoko battery.
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Diffstat (limited to 'drivers/power/supply/bq27xxx_battery.c')
| -rw-r--r-- | drivers/power/supply/bq27xxx_battery.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c index 774aa376653e..4c4a7b1c64c5 100644 --- a/drivers/power/supply/bq27xxx_battery.c +++ b/drivers/power/supply/bq27xxx_battery.c @@ -1819,7 +1819,7 @@ static int bq27xxx_battery_current(struct bq27xxx_device_info *di, if (di->opts & BQ27XXX_O_ZERO) { flags = bq27xxx_read(di, BQ27XXX_REG_FLAGS, true); - if (flags & BQ27000_FLAG_CHGS) { + if (!(flags & BQ27000_FLAG_CHGS)) { dev_dbg(di->dev, "negative current!\n"); curr = -curr; } @@ -1827,7 +1827,7 @@ static int bq27xxx_battery_current(struct bq27xxx_device_info *di, val->intval = curr * BQ27XXX_CURRENT_CONSTANT / BQ27XXX_RS; } else { /* Other gauges return signed value */ - val->intval = (int)((s16)curr) * 1000; + val->intval = -(int)((s16)curr) * 1000; } return 0; |