rtc: ds1742: clean up ds1742_nvram_read()/ds1742_nvram_write()
authorVladimir Zapolskiy <vz@mleia.com>
Sun, 26 Jul 2015 21:48:32 +0000 (00:48 +0300)
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>
Sat, 5 Sep 2015 11:19:10 +0000 (13:19 +0200)
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>
drivers/rtc/rtc-ds1742.c

index 0f8d8ace15156fb82cd0c51394eb28a5b6ed0b5c..c5168b3bcf1a6024cb60bc48fa4f7dfca1476dc7 100644 (file)
@@ -134,7 +134,7 @@ static ssize_t ds1742_nvram_read(struct file *filp, struct kobject *kobj,
        void __iomem *ioaddr = pdata->ioaddr_nvram;
        ssize_t count;
 
-       for (count = 0; size > 0 && pos < pdata->size_nvram; count++, size--)
+       for (count = 0; count < size; count++)
                *buf++ = readb(ioaddr + pos++);
        return count;
 }
@@ -149,7 +149,7 @@ static ssize_t ds1742_nvram_write(struct file *filp, struct kobject *kobj,
        void __iomem *ioaddr = pdata->ioaddr_nvram;
        ssize_t count;
 
-       for (count = 0; size > 0 && pos < pdata->size_nvram; count++, size--)
+       for (count = 0; count < size; count++)
                writeb(*buf++, ioaddr + pos++);
        return count;
 }