From db15e6312efd537e2deb2cbad110c23f98704a3c Mon Sep 17 00:00:00 2001 From: Marek Belisko Date: Fri, 30 Aug 2013 17:47:57 -0700 Subject: [PATCH] rx51_battery: Fix channel number when reading adc value This issue was introduced in commit 8e2747f031bd "power: rx51_battery: Replace hardcoded channels values." Original code use channel as argument which was shifted by one in function. After mentioned commit argument is already shifted so we need to get index back. Signed-off-by: Marek Belisko Signed-off-by: Anton Vorontsov --- drivers/power/rx51_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/rx51_battery.c b/drivers/power/rx51_battery.c index 03f5761..1bc5857 100644 --- a/drivers/power/rx51_battery.c +++ b/drivers/power/rx51_battery.c @@ -51,7 +51,7 @@ static int rx51_battery_read_adc(int channel) if (twl4030_madc_conversion(&req) <= 0) return -ENODATA; - return req.rbuf[channel]; + return req.rbuf[ffs(channel) - 1]; } /* -- 2.7.4