aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/rtc
diff options
context:
space:
mode:
authorVladimir Zapolskiy <vz@mleia.com>2015-07-26 17:48:35 -0400
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>2015-09-05 07:19:10 -0400
commitecc663c3d0dc181fd15e4d60753024d180fd7e8f (patch)
tree8616de1f3e594e4cc273c33d67d588e227dfca20 /drivers/rtc
parentd1cf4bdee8e574c0d2b7ce9fa8d9cd89c0ac761a (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.c4
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}