mirror of
https://github.com/torvalds/linux.git
synced 2024-11-15 00:21:59 +00:00
bq27x00_battery: Fix flag register read
When reading flags, bq27x00_read() argument is inverted and causes reads 2 of bytes for bq27200 and 1 byte for bq27500, while their register sizes are 1 and 2 bytes respectively. This causes bq27500 upper flag bits always to be returned as 0, causing full charge state to never be reported correctly, so fix it. Cc: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Grazvydas Ignotas <notasas@gmail.com> Acked-by: Pali Rohár <pali.rohar@gmail.com> Signed-off-by: Anton Vorontsov <cbouatmailru@gmail.com>
This commit is contained in:
parent
b7aaacf56a
commit
4d4036591b
@ -311,7 +311,7 @@ static void bq27x00_update(struct bq27x00_device_info *di)
|
||||
struct bq27x00_reg_cache cache = {0, };
|
||||
bool is_bq27500 = di->chip == BQ27500;
|
||||
|
||||
cache.flags = bq27x00_read(di, BQ27x00_REG_FLAGS, is_bq27500);
|
||||
cache.flags = bq27x00_read(di, BQ27x00_REG_FLAGS, !is_bq27500);
|
||||
if (cache.flags >= 0) {
|
||||
if (!is_bq27500 && (cache.flags & BQ27000_FLAG_CI)) {
|
||||
dev_info(di->dev, "battery is not calibrated! ignoring capacity values\n");
|
||||
|
Loading…
Reference in New Issue
Block a user