diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2012-11-20 13:59:00 +0400 |
---|---|---|
committer | Bryan Wu <cooloney@gmail.com> | 2012-11-27 02:43:54 +0400 |
commit | 313e8b50c89a76c4f36af527f937c2f8cf4cad41 (patch) | |
tree | 01f19fbd21b2243fe5aa9106d65e82def8c5d400 /drivers/leds | |
parent | eef4aa652c40237f73258d5418b03d60a2ffec40 (diff) | |
download | linux-313e8b50c89a76c4f36af527f937c2f8cf4cad41.tar.xz |
leds: lp5521: Fix return value
Return the value obtained from i2c_smbus_read_byte_data() instead of
-EIO. Silences the following smatch warning:
drivers/leds/leds-lp5521.c:155 lp5521_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/leds')
-rw-r--r-- | drivers/leds/leds-lp5521.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/leds/leds-lp5521.c b/drivers/leds/leds-lp5521.c index 362e29e27db9..8505afe4b52e 100644 --- a/drivers/leds/leds-lp5521.c +++ b/drivers/leds/leds-lp5521.c @@ -152,7 +152,7 @@ static int lp5521_read(struct i2c_client *client, u8 reg, u8 *buf) ret = i2c_smbus_read_byte_data(client, reg); if (ret < 0) - return -EIO; + return ret; *buf = ret; return 0; |