Fix for: drivers/staging/iio/gyro/adis16080_core.c: In function ‘adis16080_read_raw’: drivers/staging/iio/gyro/adis16080_core.c:99:8: warning: ‘ut’ may be used uninitialized in this function [-Wuninitialized] Initialize ut and change error handling from adis16080_read_raw(). Signed-off-by: Danny Kukawka <danny.kukawka@xxxxxxxxx> --- drivers/staging/iio/gyro/adis16080_core.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/gyro/adis16080_core.c b/drivers/staging/iio/gyro/adis16080_core.c index 1815490..e0b2a29 100644 --- a/drivers/staging/iio/gyro/adis16080_core.c +++ b/drivers/staging/iio/gyro/adis16080_core.c @@ -82,7 +82,7 @@ static int adis16080_read_raw(struct iio_dev *indio_dev, long mask) { int ret = -EINVAL; - u16 ut; + u16 ut = 0; /* Take the iio_dev status lock */ mutex_lock(&indio_dev->mlock); @@ -94,7 +94,7 @@ static int adis16080_read_raw(struct iio_dev *indio_dev, if (ret < 0) break; ret = adis16080_spi_read(indio_dev, &ut); - if (ret < 0) + if (ret) break; *val = ut; ret = IIO_VAL_INT; -- 1.7.7.3 -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html