| From 4f5434086d9223f20b3128a7dc78b35271e76655 Mon Sep 17 00:00:00 2001 |
| From: Linus Walleij <linus.walleij@linaro.org> |
| Date: Thu, 24 Dec 2020 02:17:00 +0100 |
| Subject: iio: adc: ab8500-gpadc: Fix off by 10 to 3 |
| |
| From: Linus Walleij <linus.walleij@linaro.org> |
| |
| commit 4f5434086d9223f20b3128a7dc78b35271e76655 upstream. |
| |
| Fix an off by three orders of magnitude error in the AB8500 |
| GPADC driver. Luckily it showed up quite quickly when trying |
| to make use of it. The processed reads were returning |
| microvolts, microamperes and microcelsius instead of millivolts, |
| milliamperes and millicelsius as advertised. |
| |
| Cc: stable@vger.kernel.org |
| Fixes: 07063bbfa98e ("iio: adc: New driver for the AB8500 GPADC") |
| Signed-off-by: Linus Walleij <linus.walleij@linaro.org> |
| Link: https://lore.kernel.org/r/20201224011700.1059659-1-linus.walleij@linaro.org |
| Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/iio/adc/ab8500-gpadc.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/iio/adc/ab8500-gpadc.c |
| +++ b/drivers/iio/adc/ab8500-gpadc.c |
| @@ -918,7 +918,7 @@ static int ab8500_gpadc_read_raw(struct |
| return processed; |
| |
| /* Return millivolt or milliamps or millicentigrades */ |
| - *val = processed * 1000; |
| + *val = processed; |
| return IIO_VAL_INT; |
| } |
| |