diff options
| author | Jonathan Bakker <xc-racer2@live.ca> | 2020-05-04 15:12:58 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-10-01 13:14:43 +0200 |
| commit | da95fdc3c756d48dc2a9ae2d32a191646b9c9813 (patch) | |
| tree | a626544fbbc9635ec0e0e9f11ed42e299d7f0cc6 | |
| parent | 318af7241223eea9fc16413b04a6915518ab1e9c (diff) | |
| download | linux-stable-da95fdc3c756d48dc2a9ae2d32a191646b9c9813.tar.gz linux-stable-da95fdc3c756d48dc2a9ae2d32a191646b9c9813.tar.bz2 linux-stable-da95fdc3c756d48dc2a9ae2d32a191646b9c9813.zip | |
power: supply: max17040: Correct voltage reading
[ Upstream commit 0383024f811aa469df258039807810fc3793a105 ]
According to the datasheet available at (1), the bottom four
bits are always zero and the actual voltage is 1.25x this value
in mV. Since the kernel API specifies that voltages should be in
uV, it should report 1250x the shifted value.
1) https://datasheets.maximintegrated.com/en/ds/MAX17040-MAX17041.pdf
Signed-off-by: Jonathan Bakker <xc-racer2@live.ca>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | drivers/power/supply/max17040_battery.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/power/supply/max17040_battery.c b/drivers/power/supply/max17040_battery.c index 33c40f79d23d..2c35c13ad546 100644 --- a/drivers/power/supply/max17040_battery.c +++ b/drivers/power/supply/max17040_battery.c @@ -109,7 +109,7 @@ static void max17040_get_vcell(struct i2c_client *client) vcell = max17040_read_reg(client, MAX17040_VCELL); - chip->vcell = vcell; + chip->vcell = (vcell >> 4) * 1250; } static void max17040_get_soc(struct i2c_client *client) |