Commit f2112259 authored by Dan Carpenter's avatar Dan Carpenter Committed by Jonathan Cameron

iio: light: apds9960: silence uninitialized variable warning

It causes a static checker warning if we use "buf" on the failure path
so move that inside the if statement.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent eb379846
...@@ -495,9 +495,10 @@ static int apds9960_read_raw(struct iio_dev *indio_dev, ...@@ -495,9 +495,10 @@ static int apds9960_read_raw(struct iio_dev *indio_dev,
case IIO_INTENSITY: case IIO_INTENSITY:
ret = regmap_bulk_read(data->regmap, chan->address, ret = regmap_bulk_read(data->regmap, chan->address,
&buf, 2); &buf, 2);
if (!ret) if (!ret) {
ret = IIO_VAL_INT; ret = IIO_VAL_INT;
*val = le16_to_cpu(buf); *val = le16_to_cpu(buf);
}
break; break;
default: default:
ret = -EINVAL; ret = -EINVAL;
......
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