diff options
author | Naidu Tellapati <naidu.tellapati@imgtec.com> | 2015-05-07 17:22:19 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2015-05-12 15:01:12 -0400 |
commit | 65a761bf8d55fdcf8ecc4642382a4e76c086e44c (patch) | |
tree | 6343cad1346a853f9aebf18fbd5b0c734d53a1d1 /drivers/iio | |
parent | 713276ea88a295a79aa6633ba639ed47692a8de4 (diff) |
iio: adc: cc10001: Fix regulator_get_voltage() return value check
regulator_get_voltage() returns a non-negative value in case of success,
and a negative error in case of error. Let's fix this.
Fixes: 1664f6a5b0c8 ("iio: adc: Cosmic Circuits 10001 ADC driver")
Signed-off-by: Naidu Tellapati <naidu.tellapati@imgtec.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@imgtec.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio')
-rw-r--r-- | drivers/iio/adc/cc10001_adc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/adc/cc10001_adc.c b/drivers/iio/adc/cc10001_adc.c index e7810ac8e295..fb7c5d2cc61a 100644 --- a/drivers/iio/adc/cc10001_adc.c +++ b/drivers/iio/adc/cc10001_adc.c | |||
@@ -230,7 +230,7 @@ static int cc10001_adc_read_raw(struct iio_dev *indio_dev, | |||
230 | 230 | ||
231 | case IIO_CHAN_INFO_SCALE: | 231 | case IIO_CHAN_INFO_SCALE: |
232 | ret = regulator_get_voltage(adc_dev->reg); | 232 | ret = regulator_get_voltage(adc_dev->reg); |
233 | if (ret) | 233 | if (ret < 0) |
234 | return ret; | 234 | return ret; |
235 | 235 | ||
236 | *val = ret / 1000; | 236 | *val = ret / 1000; |