leds: lp3944: Fix return value
authorSachin Kamat <sachin.kamat@linaro.org>
Tue, 20 Nov 2012 09:59:02 +0000 (01:59 -0800)
committerBryan Wu <cooloney@gmail.com>
Mon, 26 Nov 2012 22:43:55 +0000 (14:43 -0800)
Return the value obtained from i2c_smbus_read_byte_data() instead of
-EINVAL. Silences the following smatch warning:
drivers/leds/leds-lp3944.c:89 lp3944_reg_read() info:
why not propagate 'tmp' from i2c_smbus_read_byte_data() instead of -22?

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
drivers/leds/leds-lp3944.c

index c298f7d..b2d82e2 100644 (file)
@@ -86,7 +86,7 @@ static int lp3944_reg_read(struct i2c_client *client, u8 reg, u8 *value)
 
        tmp = i2c_smbus_read_byte_data(client, reg);
        if (tmp < 0)
-               return -EINVAL;
+               return tmp;
 
        *value = tmp;