hwmon: (i5k_amb) fix checkpatch issues
authorFrans Meulenbroeks <fransmeulenbroeks@gmail.com>
Sun, 8 Jan 2012 18:34:20 +0000 (19:34 +0100)
committerGuenter Roeck <guenter.roeck@ericsson.com>
Mon, 19 Mar 2012 01:26:30 +0000 (18:26 -0700)
fixed:
WARNING: simple_strtoul is obsolete, use kstrtoul instead
+ unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;

WARNING: simple_strtoul is obsolete, use kstrtoul instead
+ unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;

WARNING: simple_strtoul is obsolete, use kstrtoul instead
+ unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;

Signed-off-by: Frans Meulenbroeks <fransmeulenbroeks@gmail.com>
Acked-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/i5k_amb.c

index d22f241b6a67ea8b1f2030a54a913360db615a62..a18882cc073d34f0318afea85c65ed7b813a000c 100644 (file)
@@ -159,8 +159,12 @@ static ssize_t store_amb_min(struct device *dev,
 {
        struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
        struct i5k_amb_data *data = dev_get_drvdata(dev);
-       unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;
+       unsigned long temp;
+       int ret = kstrtoul(buf, 10, &temp);
+       if (ret < 0)
+               return ret;
 
+       temp = temp / 500;
        if (temp > 255)
                temp = 255;
 
@@ -175,8 +179,12 @@ static ssize_t store_amb_mid(struct device *dev,
 {
        struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
        struct i5k_amb_data *data = dev_get_drvdata(dev);
-       unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;
+       unsigned long temp;
+       int ret = kstrtoul(buf, 10, &temp);
+       if (ret < 0)
+               return ret;
 
+       temp = temp / 500;
        if (temp > 255)
                temp = 255;
 
@@ -191,8 +199,12 @@ static ssize_t store_amb_max(struct device *dev,
 {
        struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
        struct i5k_amb_data *data = dev_get_drvdata(dev);
-       unsigned long temp = simple_strtoul(buf, NULL, 10) / 500;
+       unsigned long temp;
+       int ret = kstrtoul(buf, 10, &temp);
+       if (ret < 0)
+               return ret;
 
+       temp = temp / 500;
        if (temp > 255)
                temp = 255;