diff options
| author | Felix Fietkau <nbd@openwrt.org> | 2016-02-25 13:31:23 +0000 |
|---|---|---|
| committer | Felix Fietkau <nbd@openwrt.org> | 2016-02-25 13:31:23 +0000 |
| commit | 3b16a635239718f94c2bef5bd0ee9f043f1451e0 (patch) | |
| tree | e2d9feea3e22e49863947dfe34d8584e76eea075 /target/linux/generic/files | |
| parent | 1fd65f01a6377deee9cd0a96a24079c8219663c3 (diff) | |
| download | master-187ad058-3b16a635239718f94c2bef5bd0ee9f043f1451e0.tar.gz master-187ad058-3b16a635239718f94c2bef5bd0ee9f043f1451e0.tar.bz2 master-187ad058-3b16a635239718f94c2bef5bd0ee9f043f1451e0.zip | |
switch: make LED port_mask file write handler use kstrtoul() function
This patch changes swconfig_trig_port_mask_store() handler to utilize
kstrtoul() function instead of call to obsolete simple_strtoul(). Thanks
to this change, new handler takes less memory and makes port_mask special
file accept not only hexadecimal, but also decimal and octal numbers.
Signed-off-by: Michal Cieslakiewicz <michal.cieslakiewicz@wp.pl>
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@48774 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/generic/files')
| -rw-r--r-- | target/linux/generic/files/drivers/net/phy/swconfig_leds.c | 41 |
1 files changed, 16 insertions, 25 deletions
diff --git a/target/linux/generic/files/drivers/net/phy/swconfig_leds.c b/target/linux/generic/files/drivers/net/phy/swconfig_leds.c index abd7bed48c..7d122d26cd 100644 --- a/target/linux/generic/files/drivers/net/phy/swconfig_leds.c +++ b/target/linux/generic/files/drivers/net/phy/swconfig_leds.c @@ -94,37 +94,28 @@ swconfig_trig_port_mask_store(struct device *dev, struct device_attribute *attr, struct led_classdev *led_cdev = dev_get_drvdata(dev); struct swconfig_trig_data *trig_data = led_cdev->trigger_data; unsigned long port_mask; - ssize_t ret = -EINVAL; - char *after; - size_t count; + int ret; + bool changed; - port_mask = simple_strtoul(buf, &after, 16); - count = after - buf; + ret = kstrtoul(buf, 0, &port_mask); + if (ret) + return ret; - if (*after && isspace(*after)) - count++; + write_lock(&trig_data->lock); - if (count == size) { - bool changed; - - write_lock(&trig_data->lock); - - changed = (trig_data->port_mask != port_mask); - if (changed) { - trig_data->port_mask = port_mask; - if (port_mask == 0) - swconfig_trig_set_brightness(trig_data, LED_OFF); - } - - write_unlock(&trig_data->lock); + changed = (trig_data->port_mask != port_mask); + if (changed) { + trig_data->port_mask = port_mask; + if (port_mask == 0) + swconfig_trig_set_brightness(trig_data, LED_OFF); + } - if (changed) - swconfig_trig_update_port_mask(led_cdev->trigger); + write_unlock(&trig_data->lock); - ret = count; - } + if (changed) + swconfig_trig_update_port_mask(led_cdev->trigger); - return ret; + return size; } static ssize_t |