| From 84f44aea924f500c1000499bf9d3fe10e26742c4 Mon Sep 17 00:00:00 2001 |
| From: Eva Rachel Retuya <eraretuya@gmail.com> |
| Date: Sun, 9 Oct 2016 00:05:39 +0800 |
| Subject: [PATCH] staging: iio: ad7606: fix improper setting of oversampling |
| pins |
| |
| commit b321a38d2407c7e425c54bc09be909a34e49f740 upstream. |
| |
| The oversampling ratio is controlled using the oversampling pins, |
| OS [2:0] with OS2 being the MSB control bit, and OS0 the LSB control |
| bit. |
| |
| The gpio connected to the OS2 pin is not being set correctly, only OS0 |
| and OS1 pins are being set. Fix the typo to allow proper control of the |
| oversampling pins. |
| |
| Signed-off-by: Eva Rachel Retuya <eraretuya@gmail.com> |
| Fixes: b9618c0 ("staging: IIO: ADC: New driver for AD7606/AD7606-6/AD7606-4") |
| Acked-by: Lars-Peter Clausen <lars@metafoo.de> |
| Signed-off-by: Jonathan Cameron <jic23@kernel.org> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c |
| index f79ee61851f6..cbd6bc52050f 100644 |
| --- a/drivers/staging/iio/adc/ad7606_core.c |
| +++ b/drivers/staging/iio/adc/ad7606_core.c |
| @@ -189,7 +189,7 @@ static ssize_t ad7606_store_oversampling_ratio(struct device *dev, |
| mutex_lock(&indio_dev->mlock); |
| gpio_set_value(st->pdata->gpio_os0, (ret >> 0) & 1); |
| gpio_set_value(st->pdata->gpio_os1, (ret >> 1) & 1); |
| - gpio_set_value(st->pdata->gpio_os1, (ret >> 2) & 1); |
| + gpio_set_value(st->pdata->gpio_os2, (ret >> 2) & 1); |
| st->oversampling = lval; |
| mutex_unlock(&indio_dev->mlock); |
| |
| -- |
| 2.10.1 |
| |