Commit ecc663c3 authored by Vladimir Zapolskiy's avatar Vladimir Zapolskiy Committed by Alexandre Belloni

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: default avatarVladimir Zapolskiy <vz@mleia.com>
Signed-off-by: default avatarAlexandre Belloni <alexandre.belloni@free-electrons.com>
parent d1cf4bde
...@@ -254,7 +254,7 @@ static ssize_t stk17ta8_nvram_read(struct file *filp, struct kobject *kobj, ...@@ -254,7 +254,7 @@ static ssize_t stk17ta8_nvram_read(struct file *filp, struct kobject *kobj,
void __iomem *ioaddr = pdata->ioaddr; void __iomem *ioaddr = pdata->ioaddr;
ssize_t count; ssize_t count;
for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) for (count = 0; count < size; count++)
*buf++ = readb(ioaddr + pos++); *buf++ = readb(ioaddr + pos++);
return count; return count;
} }
...@@ -269,7 +269,7 @@ static ssize_t stk17ta8_nvram_write(struct file *filp, struct kobject *kobj, ...@@ -269,7 +269,7 @@ static ssize_t stk17ta8_nvram_write(struct file *filp, struct kobject *kobj,
void __iomem *ioaddr = pdata->ioaddr; void __iomem *ioaddr = pdata->ioaddr;
ssize_t count; ssize_t count;
for (count = 0; size > 0 && pos < RTC_OFFSET; count++, size--) for (count = 0; count < size; count++)
writeb(*buf++, ioaddr + pos++); writeb(*buf++, ioaddr + pos++);
return count; return count;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment