summaryrefslogtreecommitdiffstats
path: root/drivers/iio/adc
diff options
context:
space:
mode:
authorBeniamin Bia <beniamin.bia@analog.com>2019-11-04 17:26:34 +0100
committerJonathan Cameron <Jonathan.Cameron@huawei.com>2019-12-08 19:07:40 +0100
commit341826a065660d1b77d89e6335b6095cd654271c (patch)
treef494e21280beee5787763d0593b2bde169226f0a /drivers/iio/adc
parentstaging: vchiq: Refactor indentation in vchiq_platform_conn_state_changed() (diff)
downloadlinux-341826a065660d1b77d89e6335b6095cd654271c.tar.xz
linux-341826a065660d1b77d89e6335b6095cd654271c.zip
iio: adc: ad7606: fix reading unnecessary data from device
When a conversion result is being read from ADC, the driver reads the number of channels + 1 because it thinks that IIO_CHAN_SOFT_TIMESTAMP is also a physical channel. This patch fixes this issue. Fixes: 2985a5d88455 ("staging: iio: adc: ad7606: Move out of staging") Reported-by: Robert Wörle <rwoerle@mibtec.de> Signed-off-by: Beniamin Bia <beniamin.bia@analog.com> Cc: <Stable@vger.kernel.org> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio/adc')
-rw-r--r--drivers/iio/adc/ad7606.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/adc/ad7606.c b/drivers/iio/adc/ad7606.c
index f5ba94c03a8d..e4683a68522a 100644
--- a/drivers/iio/adc/ad7606.c
+++ b/drivers/iio/adc/ad7606.c
@@ -85,7 +85,7 @@ err_unlock:
static int ad7606_read_samples(struct ad7606_state *st)
{
- unsigned int num = st->chip_info->num_channels;
+ unsigned int num = st->chip_info->num_channels - 1;
u16 *data = st->data;
int ret;