diff options
| author | Darren Hart <dvhart@linux.intel.com> | 2015-02-07 03:49:23 +0100 |
|---|---|---|
| committer | Darren Hart <dvhart@linux.intel.com> | 2015-02-07 03:53:59 +0100 |
| commit | 802cf2e1e0f202507b6e21b7a12da56d5c86af62 (patch) | |
| tree | addb20e83ff037c6d5ed69a13572dc3d074c128a /drivers/platform | |
| parent | samsung-laptop: Fix sparse integer as NULL warning (diff) | |
| download | linux-802cf2e1e0f202507b6e21b7a12da56d5c86af62.tar.xz linux-802cf2e1e0f202507b6e21b7a12da56d5c86af62.zip | |
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>
Diffstat (limited to 'drivers/platform')
| -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, struct samsung_laptop *samsung = dev_get_drvdata(dev); int ret, value; - if (!count || sscanf(buf, "%i", &value) != 1) + if (!count || kstrtoint(buf, 0, &value) != 0) return -EINVAL; ret = write_battery_life_extender(samsung, !!value); @@ -837,7 +837,7 @@ static ssize_t set_usb_charge(struct device *dev, struct samsung_laptop *samsung = dev_get_drvdata(dev); int ret, value; - if (!count || sscanf(buf, "%i", &value) != 1) + if (!count || kstrtoint(buf, 0, &value) != 0) return -EINVAL; ret = write_usb_charge(samsung, !!value); |