diff options
author | JJ Ding <dgdunix@gmail.com> | 2011-11-09 13:20:14 -0500 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2011-11-10 00:23:26 -0500 |
commit | 76496e7a02e99d42844f4fffa145b81e513e7acd (patch) | |
tree | 33812cc8a9b250a95cf90c237c46ec6fc6fcf2ff /drivers/input/misc/ati_remote2.c | |
parent | 7cf801cfc0774b777aa6861cf4a43a90b112b1ed (diff) |
Input: convert obsolete strict_strtox to kstrtox
With commit 67d0a0754455f89ef3946946159d8ec9e45ce33a we mark strict_strtox
as obsolete. Convert all remaining such uses in drivers/input/.
Also change long to appropriate types, and return error conditions
from kstrtox separately, as Dmitry sugguests.
Signed-off-by: JJ Ding <dgdunix@gmail.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/input/misc/ati_remote2.c')
-rw-r--r-- | drivers/input/misc/ati_remote2.c | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/drivers/input/misc/ati_remote2.c b/drivers/input/misc/ati_remote2.c index 1de58e8a1b71..afbe3e760551 100644 --- a/drivers/input/misc/ati_remote2.c +++ b/drivers/input/misc/ati_remote2.c | |||
@@ -41,13 +41,13 @@ static int ati_remote2_set_mask(const char *val, | |||
41 | const struct kernel_param *kp, | 41 | const struct kernel_param *kp, |
42 | unsigned int max) | 42 | unsigned int max) |
43 | { | 43 | { |
44 | unsigned long mask; | 44 | unsigned int mask; |
45 | int ret; | 45 | int ret; |
46 | 46 | ||
47 | if (!val) | 47 | if (!val) |
48 | return -EINVAL; | 48 | return -EINVAL; |
49 | 49 | ||
50 | ret = strict_strtoul(val, 0, &mask); | 50 | ret = kstrtouint(val, 0, &mask); |
51 | if (ret) | 51 | if (ret) |
52 | return ret; | 52 | return ret; |
53 | 53 | ||
@@ -719,11 +719,12 @@ static ssize_t ati_remote2_store_channel_mask(struct device *dev, | |||
719 | struct usb_device *udev = to_usb_device(dev); | 719 | struct usb_device *udev = to_usb_device(dev); |
720 | struct usb_interface *intf = usb_ifnum_to_if(udev, 0); | 720 | struct usb_interface *intf = usb_ifnum_to_if(udev, 0); |
721 | struct ati_remote2 *ar2 = usb_get_intfdata(intf); | 721 | struct ati_remote2 *ar2 = usb_get_intfdata(intf); |
722 | unsigned long mask; | 722 | unsigned int mask; |
723 | int r; | 723 | int r; |
724 | 724 | ||
725 | if (strict_strtoul(buf, 0, &mask)) | 725 | r = kstrtouint(buf, 0, &mask); |
726 | return -EINVAL; | 726 | if (r) |
727 | return r; | ||
727 | 728 | ||
728 | if (mask & ~ATI_REMOTE2_MAX_CHANNEL_MASK) | 729 | if (mask & ~ATI_REMOTE2_MAX_CHANNEL_MASK) |
729 | return -EINVAL; | 730 | return -EINVAL; |
@@ -768,10 +769,12 @@ static ssize_t ati_remote2_store_mode_mask(struct device *dev, | |||
768 | struct usb_device *udev = to_usb_device(dev); | 769 | struct usb_device *udev = to_usb_device(dev); |
769 | struct usb_interface *intf = usb_ifnum_to_if(udev, 0); | 770 | struct usb_interface *intf = usb_ifnum_to_if(udev, 0); |
770 | struct ati_remote2 *ar2 = usb_get_intfdata(intf); | 771 | struct ati_remote2 *ar2 = usb_get_intfdata(intf); |
771 | unsigned long mask; | 772 | unsigned int mask; |
773 | int err; | ||
772 | 774 | ||
773 | if (strict_strtoul(buf, 0, &mask)) | 775 | err = kstrtouint(buf, 0, &mask); |
774 | return -EINVAL; | 776 | if (err) |
777 | return err; | ||
775 | 778 | ||
776 | if (mask & ~ATI_REMOTE2_MAX_MODE_MASK) | 779 | if (mask & ~ATI_REMOTE2_MAX_MODE_MASK) |
777 | return -EINVAL; | 780 | return -EINVAL; |