Messages in this thread Patch in this message | | | From | Danny Kukawka <> | Subject | [PATCH] adis16080: fix compiler -Wuninitialized | Date | Tue, 14 Feb 2012 15:35:36 +0100 |
| |
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@bisect.de> --- 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-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |