diff options
author | Vladimir Zapolskiy <vz@mleia.com> | 2015-07-26 17:48:35 -0400 |
---|---|---|
committer | Alexandre Belloni <alexandre.belloni@free-electrons.com> | 2015-09-05 07:19:10 -0400 |
commit | ecc663c3d0dc181fd15e4d60753024d180fd7e8f (patch) | |
tree | 8616de1f3e594e4cc273c33d67d588e227dfca20 /drivers/rtc | |
parent | d1cf4bdee8e574c0d2b7ce9fa8d9cd89c0ac761a (diff) |
rtc: stk17ta8: clean up stk17ta8_nvram_read()/stk17ta8_nvram_write()
The change removes redundant sysfs binary file boundary checks, since
this task is already done on caller side in fs/sysfs/file.c
Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Diffstat (limited to 'drivers/rtc')
-rw-r--r-- | drivers/rtc/rtc-stk17ta8.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/rtc/rtc-stk17ta8.c b/drivers/rtc/rtc-stk17ta8.c index 0e93b714ee41..ba6a83b5b5c9 100644 --- a/drivers/rtc/rtc-stk17ta8.c +++ b/drivers/rtc/rtc-stk17ta8.c | |||
@@ -254,7 +254,7 @@ static ssize_t stk17ta8_nvram_read(struct file *filp, struct kobject *kobj, | |||
254 | void __iomem *ioaddr = pdata->ioaddr; | 254 | void __iomem *ioaddr = pdata->ioaddr; |
255 | ssize_t count; | 255 | ssize_t count; |
256 | 256 | ||
257 | for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) | 257 | for (count = 0; count < size; count++) |
258 | *buf++ = readb(ioaddr + pos++); | 258 | *buf++ = readb(ioaddr + pos++); |
259 | return count; | 259 | return count; |
260 | } | 260 | } |
@@ -269,7 +269,7 @@ static ssize_t stk17ta8_nvram_write(struct file *filp, struct kobject *kobj, | |||
269 | void __iomem *ioaddr = pdata->ioaddr; | 269 | void __iomem *ioaddr = pdata->ioaddr; |
270 | ssize_t count; | 270 | ssize_t count; |
271 | 271 | ||
272 | for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) | 272 | for (count = 0; count < size; count++) |
273 | writeb(*buf++, ioaddr + pos++); | 273 | writeb(*buf++, ioaddr + pos++); |
274 | return count; | 274 | return count; |
275 | } | 275 | } |