diff options
| author | Sachin Kamat | 2012-11-20 01:59:00 -0800 |
|---|---|---|
| committer | Bryan Wu | 2012-11-26 14:43:54 -0800 |
| commit | 313e8b50c89a76c4f36af527f937c2f8cf4cad41 (patch) | |
| tree | 01f19fbd21b2243fe5aa9106d65e82def8c5d400 /drivers | |
| parent | eef4aa652c40237f73258d5418b03d60a2ffec40 (diff) | |
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')
| -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; |