summaryrefslogtreecommitdiffstats
path: root/drivers/iio/addac
diff options
context:
space:
mode:
authorJonathan Cameron <Jonathan.Cameron@huawei.com>2021-12-20 16:47:26 +0000
committerJonathan Cameron <Jonathan.Cameron@huawei.com>2021-12-21 15:10:10 +0000
commitf4a73a97accf5635815de148cf077fa6d076812d (patch)
tree27bb86d5e3636f34bb9a5888b9b09150dc3753fb /drivers/iio/addac
parentbfcacdd64df8df91311449642da70a8512431fa9 (diff)
downloadlinux-f4a73a97accf5635815de148cf077fa6d076812d.tar.bz2
iio:addac:ad74413r: Fix uninitialized ret in a path that won't be hit.
I don't believe it's possible to hit this, because we drop out of __iio_update_buffers() earlier in the event of an empty list. However, that is not visible to the compiler so lets return an error if we do hit the loop with an empty bitmask. Fixes: 5d97d9e9a703 ("iio: addac: ad74413r: fix off by one in ad74413r_parse_channel_config()") Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Cc: Cosmin Tanislav <cosmin.tanislav@analog.com> Link: https://lore.kernel.org/r/20211220164726.3136307-1-jic23@kernel.org
Diffstat (limited to 'drivers/iio/addac')
-rw-r--r--drivers/iio/addac/ad74413r.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/addac/ad74413r.c b/drivers/iio/addac/ad74413r.c
index 289d254943e1..5271073bb74e 100644
--- a/drivers/iio/addac/ad74413r.c
+++ b/drivers/iio/addac/ad74413r.c
@@ -843,7 +843,7 @@ static int ad74413r_update_scan_mode(struct iio_dev *indio_dev,
u8 *rx_buf = &st->adc_samples_buf.rx_buf[-1 * AD74413R_FRAME_SIZE];
u8 *tx_buf = st->adc_samples_tx_buf;
unsigned int channel;
- int ret;
+ int ret = -EINVAL;
mutex_lock(&st->lock);