On Mon, 3 Feb 2020 10:17:01 +0100 Guido Günther <agx@xxxxxxxxxxx> wrote: > This will be useful when introducing runtime pm. > > Signed-off-by: Guido Günther <agx@xxxxxxxxxxx> Applied. Thanks, Jonathan > --- > drivers/iio/light/vcnl4000.c | 15 ++++++++------- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/drivers/iio/light/vcnl4000.c b/drivers/iio/light/vcnl4000.c > index b0e241aaefb4..8f198383626b 100644 > --- a/drivers/iio/light/vcnl4000.c > +++ b/drivers/iio/light/vcnl4000.c > @@ -335,17 +335,18 @@ static int vcnl4000_read_raw(struct iio_dev *indio_dev, > switch (chan->type) { > case IIO_LIGHT: > ret = data->chip_spec->measure_light(data, val); > - if (ret < 0) > - return ret; > - return IIO_VAL_INT; > + if (!ret) > + ret = IIO_VAL_INT; > + break; > case IIO_PROXIMITY: > ret = data->chip_spec->measure_proximity(data, val); > - if (ret < 0) > - return ret; > - return IIO_VAL_INT; > + if (!ret) > + ret = IIO_VAL_INT; > + break; > default: > - return -EINVAL; > + ret = -EINVAL; > } > + return ret; > case IIO_CHAN_INFO_SCALE: > if (chan->type != IIO_LIGHT) > return -EINVAL;