diff options
author | Darren Hart <dvhart@linux.intel.com> | 2015-02-06 21:49:23 -0500 |
---|---|---|
committer | Darren Hart <dvhart@linux.intel.com> | 2015-02-06 21:53:59 -0500 |
commit | 802cf2e1e0f202507b6e21b7a12da56d5c86af62 (patch) | |
tree | addb20e83ff037c6d5ed69a13572dc3d074c128a | |
parent | 4e7f09ad5ea98ae60a435575ae877bd68a5f2d5c (diff) |
samsung-laptop.c: Prefer kstrtoint over single variable sscanf
Replace existing usage of single variable sscanf with kstrtoint for
consistency with checkpatch warnings against such usage.
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
-rw-r--r-- | drivers/platform/x86/samsung-laptop.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/platform/x86/samsung-laptop.c b/drivers/platform/x86/samsung-laptop.c index 3257b0228613..9e701b2256f9 100644 --- a/drivers/platform/x86/samsung-laptop.c +++ b/drivers/platform/x86/samsung-laptop.c | |||
@@ -768,7 +768,7 @@ static ssize_t set_battery_life_extender(struct device *dev, | |||
768 | struct samsung_laptop *samsung = dev_get_drvdata(dev); | 768 | struct samsung_laptop *samsung = dev_get_drvdata(dev); |
769 | int ret, value; | 769 | int ret, value; |
770 | 770 | ||
771 | if (!count || sscanf(buf, "%i", &value) != 1) | 771 | if (!count || kstrtoint(buf, 0, &value) != 0) |
772 | return -EINVAL; | 772 | return -EINVAL; |
773 | 773 | ||
774 | ret = write_battery_life_extender(samsung, !!value); | 774 | ret = write_battery_life_extender(samsung, !!value); |
@@ -837,7 +837,7 @@ static ssize_t set_usb_charge(struct device *dev, | |||
837 | struct samsung_laptop *samsung = dev_get_drvdata(dev); | 837 | struct samsung_laptop *samsung = dev_get_drvdata(dev); |
838 | int ret, value; | 838 | int ret, value; |
839 | 839 | ||
840 | if (!count || sscanf(buf, "%i", &value) != 1) | 840 | if (!count || kstrtoint(buf, 0, &value) != 0) |
841 | return -EINVAL; | 841 | return -EINVAL; |
842 | 842 | ||
843 | ret = write_usb_charge(samsung, !!value); | 843 | ret = write_usb_charge(samsung, !!value); |