Staging: iio: adt7316: Add an extra check for 'ret' equals to 0

ret = 0 indicates a case of no error but no data read from
the bus which is an invalid case. This case doesn't ever happen
in reality. It should perhaps be handled for correctness though.

Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
Shreeya Patel 2018-11-17 04:21:32 +05:30 committed by Jonathan Cameron
parent 688cd642ba
commit 00426e9978

View File

@ -30,6 +30,10 @@ static int adt7316_i2c_read(void *client, u8 reg, u8 *data)
} }
ret = i2c_smbus_read_byte(client); ret = i2c_smbus_read_byte(client);
if (!ret)
return -EIO;
if (ret < 0) { if (ret < 0) {
dev_err(&cl->dev, "I2C read error\n"); dev_err(&cl->dev, "I2C read error\n");
return ret; return ret;