aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/rtc
diff options
context:
space:
mode:
authorVladimir Zapolskiy <vz@mleia.com>2015-07-26 17:48:31 -0400
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>2015-09-05 07:19:10 -0400
commita9118d77b3be366c4843446d671a52ccfddcc15c (patch)
tree74877b3224fd316cb8cb56cdadc0cff214b075e8 /drivers/rtc
parent8ccba14233000f48639aad4cab55ed7b61260db5 (diff)
rtc: ds1553: clean up ds1553_nvram_read()/ds1553_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-ds1553.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/rtc/rtc-ds1553.c b/drivers/rtc/rtc-ds1553.c
index a24e091bcb41..38422ab4ec5a 100644
--- a/drivers/rtc/rtc-ds1553.c
+++ b/drivers/rtc/rtc-ds1553.c
@@ -245,7 +245,7 @@ static ssize_t ds1553_nvram_read(struct file *filp, struct kobject *kobj,
245 void __iomem *ioaddr = pdata->ioaddr; 245 void __iomem *ioaddr = pdata->ioaddr;
246 ssize_t count; 246 ssize_t count;
247 247
248 for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) 248 for (count = 0; count < size; count++)
249 *buf++ = readb(ioaddr + pos++); 249 *buf++ = readb(ioaddr + pos++);
250 return count; 250 return count;
251} 251}
@@ -260,7 +260,7 @@ static ssize_t ds1553_nvram_write(struct file *filp, struct kobject *kobj,
260 void __iomem *ioaddr = pdata->ioaddr; 260 void __iomem *ioaddr = pdata->ioaddr;
261 ssize_t count; 261 ssize_t count;
262 262
263 for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) 263 for (count = 0; count < size; count++)
264 writeb(*buf++, ioaddr + pos++); 264 writeb(*buf++, ioaddr + pos++);
265 return count; 265 return count;
266} 266}