diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-07-03 18:07:52 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-07-03 19:07:59 -0400 |
commit | ee605e0b2e12040dc1fe88eb8ab03bde0b8f92b8 (patch) | |
tree | 4e8ed470f08b81d4a7352768ee0a175e70f3fc6a | |
parent | 23f809ee33f06877de4c4f28129c495d332cd500 (diff) |
drivers/rtc/rtc-pcf8583.c: use PTR_RET()
Use of PTR_RET() simplifies the code.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/rtc/rtc-pcf8583.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/rtc/rtc-pcf8583.c b/drivers/rtc/rtc-pcf8583.c index ab740cd63109..843a745c42f3 100644 --- a/drivers/rtc/rtc-pcf8583.c +++ b/drivers/rtc/rtc-pcf8583.c | |||
@@ -17,6 +17,7 @@ | |||
17 | #include <linux/slab.h> | 17 | #include <linux/slab.h> |
18 | #include <linux/rtc.h> | 18 | #include <linux/rtc.h> |
19 | #include <linux/init.h> | 19 | #include <linux/init.h> |
20 | #include <linux/err.h> | ||
20 | #include <linux/errno.h> | 21 | #include <linux/errno.h> |
21 | #include <linux/bcd.h> | 22 | #include <linux/bcd.h> |
22 | 23 | ||
@@ -284,10 +285,7 @@ static int pcf8583_probe(struct i2c_client *client, | |||
284 | pcf8583_driver.driver.name, | 285 | pcf8583_driver.driver.name, |
285 | &pcf8583_rtc_ops, THIS_MODULE); | 286 | &pcf8583_rtc_ops, THIS_MODULE); |
286 | 287 | ||
287 | if (IS_ERR(pcf8583->rtc)) | 288 | return PTR_RET(pcf8583->rtc); |
288 | return PTR_ERR(pcf8583->rtc); | ||
289 | |||
290 | return 0; | ||
291 | } | 289 | } |
292 | 290 | ||
293 | static const struct i2c_device_id pcf8583_id[] = { | 291 | static const struct i2c_device_id pcf8583_id[] = { |