diff options
author | Vladimir Zapolskiy <vz@mleia.com> | 2015-07-26 17:48:34 -0400 |
---|---|---|
committer | Alexandre Belloni <alexandre.belloni@free-electrons.com> | 2015-09-05 07:19:10 -0400 |
commit | d1cf4bdee8e574c0d2b7ce9fa8d9cd89c0ac761a (patch) | |
tree | d2cb50f14b9aa8fdd3d350ea4798edb688ea2314 /drivers/rtc | |
parent | 99be3e371b2ac03b936044139be10cf0c7b9cc58 (diff) |
rtc: rp5c01: clean up rp5c01_nvram_read()/rp5c01_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-rp5c01.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/rtc/rtc-rp5c01.c b/drivers/rtc/rtc-rp5c01.c index b548551f385c..026035373ae6 100644 --- a/drivers/rtc/rtc-rp5c01.c +++ b/drivers/rtc/rtc-rp5c01.c | |||
@@ -170,7 +170,7 @@ static ssize_t rp5c01_nvram_read(struct file *filp, struct kobject *kobj, | |||
170 | 170 | ||
171 | spin_lock_irq(&priv->lock); | 171 | spin_lock_irq(&priv->lock); |
172 | 172 | ||
173 | for (count = 0; size > 0 && pos < RP5C01_MODE; count++, size--) { | 173 | for (count = 0; count < size; count++) { |
174 | u8 data; | 174 | u8 data; |
175 | 175 | ||
176 | rp5c01_write(priv, | 176 | rp5c01_write(priv, |
@@ -200,7 +200,7 @@ static ssize_t rp5c01_nvram_write(struct file *filp, struct kobject *kobj, | |||
200 | 200 | ||
201 | spin_lock_irq(&priv->lock); | 201 | spin_lock_irq(&priv->lock); |
202 | 202 | ||
203 | for (count = 0; size > 0 && pos < RP5C01_MODE; count++, size--) { | 203 | for (count = 0; count < size; count++) { |
204 | u8 data = *buf++; | 204 | u8 data = *buf++; |
205 | 205 | ||
206 | rp5c01_write(priv, | 206 | rp5c01_write(priv, |