diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-11-12 18:10:26 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-11-12 22:09:27 -0500 |
commit | c91fd91d686654f8afbbbca18f5ff0a6ebc9c7a2 (patch) | |
tree | 4b0c363fe12b653afd9eabd426dedb816801a23d /drivers/rtc | |
parent | 50ccf0455e51eba51e645b3724c8e932c75774b1 (diff) |
drivers/rtc/rtc-isl1208.c: remove redundant checks
i2c_smbus_read_byte_data() returns negative errno on failure. Return the
value obtained from it directly.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Cc: Herbert Valerio Riedel <hvr@gnu.org>
Cc: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/rtc')
-rw-r--r-- | drivers/rtc/rtc-isl1208.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/rtc/rtc-isl1208.c b/drivers/rtc/rtc-isl1208.c index c016ad81767a..5a0bf7ce1033 100644 --- a/drivers/rtc/rtc-isl1208.c +++ b/drivers/rtc/rtc-isl1208.c | |||
@@ -144,11 +144,7 @@ isl1208_i2c_validate_client(struct i2c_client *client) | |||
144 | static int | 144 | static int |
145 | isl1208_i2c_get_sr(struct i2c_client *client) | 145 | isl1208_i2c_get_sr(struct i2c_client *client) |
146 | { | 146 | { |
147 | int sr = i2c_smbus_read_byte_data(client, ISL1208_REG_SR); | 147 | return i2c_smbus_read_byte_data(client, ISL1208_REG_SR); |
148 | if (sr < 0) | ||
149 | return -EIO; | ||
150 | |||
151 | return sr; | ||
152 | } | 148 | } |
153 | 149 | ||
154 | static int | 150 | static int |