diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2012-11-20 10:59:01 +0100 |
---|---|---|
committer | Bryan Wu <cooloney@gmail.com> | 2012-11-26 23:43:55 +0100 |
commit | 5ebab74a711388fc30e6d839eee79aa06d1ce0af (patch) | |
tree | 7efd3f5dfdb6f9f1ad44cd574e53e58042cbd7bf /drivers | |
parent | leds: lp5521: Fix return value (diff) | |
download | linux-5ebab74a711388fc30e6d839eee79aa06d1ce0af.tar.xz linux-5ebab74a711388fc30e6d839eee79aa06d1ce0af.zip |
leds: lp5523: Fix return value
Return the value obtained from i2c_smbus_read_byte_data() instead of
-EIO. Silences the following smatch warning:
drivers/leds/leds-lp5523.c:174 lp5523_read() info:
why not propagate 'ret' from i2c_smbus_read_byte_data() instead of -5?
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/leds/leds-lp5523.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c index 59ec383900de..eb89cbd419ca 100644 --- a/drivers/leds/leds-lp5523.c +++ b/drivers/leds/leds-lp5523.c @@ -171,7 +171,7 @@ static int lp5523_read(struct i2c_client *client, u8 reg, u8 *buf) s32 ret = i2c_smbus_read_byte_data(client, reg); if (ret < 0) - return -EIO; + return ret; *buf = ret; return 0; |