diff options
author | Vladimir Zapolskiy <vz@mleia.com> | 2015-07-26 17:48:31 -0400 |
---|---|---|
committer | Alexandre Belloni <alexandre.belloni@free-electrons.com> | 2015-09-05 07:19:10 -0400 |
commit | a9118d77b3be366c4843446d671a52ccfddcc15c (patch) | |
tree | 74877b3224fd316cb8cb56cdadc0cff214b075e8 /drivers/rtc | |
parent | 8ccba14233000f48639aad4cab55ed7b61260db5 (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.c | 4 |
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 | } |