Commit 897993fe authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Jonathan Cameron

iio: sx9500: Add missing init in sx9500_buffer_pre{en,dis}able()

drivers/iio/proximity/sx9500.c: In function ‘sx9500_buffer_preenable’:
drivers/iio/proximity/sx9500.c:682: warning: ‘ret’ may be used uninitialized in this function
drivers/iio/proximity/sx9500.c: In function ‘sx9500_buffer_predisable’:
drivers/iio/proximity/sx9500.c:706: warning: ‘ret’ may be used uninitialized in this function

If active_scan_mask is empty, it will loop once more over all channels,
doing nothing.
Signed-off-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
Reviewed-by: default avatarVlad Dogaru <vlad.dogaru@intel.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent 5d9fc0f6
...@@ -682,7 +682,7 @@ static irqreturn_t sx9500_trigger_handler(int irq, void *private) ...@@ -682,7 +682,7 @@ static irqreturn_t sx9500_trigger_handler(int irq, void *private)
static int sx9500_buffer_preenable(struct iio_dev *indio_dev) static int sx9500_buffer_preenable(struct iio_dev *indio_dev)
{ {
struct sx9500_data *data = iio_priv(indio_dev); struct sx9500_data *data = iio_priv(indio_dev);
int ret, i; int ret = 0, i;
mutex_lock(&data->mutex); mutex_lock(&data->mutex);
...@@ -706,7 +706,7 @@ static int sx9500_buffer_preenable(struct iio_dev *indio_dev) ...@@ -706,7 +706,7 @@ static int sx9500_buffer_preenable(struct iio_dev *indio_dev)
static int sx9500_buffer_predisable(struct iio_dev *indio_dev) static int sx9500_buffer_predisable(struct iio_dev *indio_dev)
{ {
struct sx9500_data *data = iio_priv(indio_dev); struct sx9500_data *data = iio_priv(indio_dev);
int ret, i; int ret = 0, i;
iio_triggered_buffer_predisable(indio_dev); iio_triggered_buffer_predisable(indio_dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment