diff options
| author | Vladimir Zapolskiy <vz@mleia.com> | 2015-07-27 00:26:49 +0300 |
|---|---|---|
| committer | Sebastian Reichel <sre@kernel.org> | 2015-07-27 16:28:54 +0200 |
| commit | 3e1d9c6f241961e2e21b8df9de5ee12e3dc6f177 (patch) | |
| tree | a4d372e6327efbca685cbb2e7f5baa8556d8485c /drivers | |
| parent | 9f28b86552aef94733387814a57755cb0019cfb9 (diff) | |
| download | linux-3e1d9c6f241961e2e21b8df9de5ee12e3dc6f177.tar.bz2 | |
power: olpc_battery: clean up eeprom read function
The change removes redundant sysfs binary file boundary check while
reading eeprom content from userspace, the check is done on caller
side in fs/sysfs/file.c, if binary attribute size is not zero.
Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/power/olpc_battery.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/power/olpc_battery.c b/drivers/power/olpc_battery.c index a944338a39de..9e29b1321648 100644 --- a/drivers/power/olpc_battery.c +++ b/drivers/power/olpc_battery.c @@ -521,11 +521,6 @@ static ssize_t olpc_bat_eeprom_read(struct file *filp, struct kobject *kobj, int ret; int i; - if (off >= EEPROM_SIZE) - return 0; - if (off + count > EEPROM_SIZE) - count = EEPROM_SIZE - off; - for (i = 0; i < count; i++) { ec_byte = EEPROM_START + off + i; ret = olpc_ec_cmd(EC_BAT_EEPROM, &ec_byte, 1, &buf[i], 1); @@ -545,7 +540,7 @@ static struct bin_attribute olpc_bat_eeprom = { .name = "eeprom", .mode = S_IRUGO, }, - .size = 0, + .size = EEPROM_SIZE, .read = olpc_bat_eeprom_read, }; |