summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2021-03-23 13:27:05 +0100
committerJonathan Cameron <Jonathan.Cameron@huawei.com>2021-04-07 08:35:51 +0100
commitb3b64e2c1575da5f290e75e2a1d9cac307812222 (patch)
tree9cea6307a5e1c1172c448008cc6b026adeb885c6
parent3e55bb6f2ac0bbd7ab8e5bde7b4f8b574afe2c52 (diff)
iio: Fix iio_read_channel_processed_scale()
The code was checking if (ret) from the processed channel readout, not smart, we need to check if (ret < 0) as this will likely be something like IIO_VAL_INT. Fixes: 635ef601b238 ("iio: Provide iio_read_channel_processed_scale() API") Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Link: https://lore.kernel.org/r/20210323122705.1326362-1-linus.walleij@linaro.org Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
-rw-r--r--drivers/iio/inkern.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/inkern.c b/drivers/iio/inkern.c
index c61fc06f98b8..9c22697b7e83 100644
--- a/drivers/iio/inkern.c
+++ b/drivers/iio/inkern.c
@@ -702,7 +702,7 @@ int iio_read_channel_processed_scale(struct iio_channel *chan, int *val,
if (iio_channel_has_info(chan->channel, IIO_CHAN_INFO_PROCESSED)) {
ret = iio_channel_read(chan, val, NULL,
IIO_CHAN_INFO_PROCESSED);
- if (ret)
+ if (ret < 0)
goto err_unlock;
*val *= scale;
} else {