hwmon: (applesmc) Fix checkpatch errors and fix value range checks
authorGuenter Roeck <guenter.roeck@ericsson.com>
Wed, 10 Nov 2010 18:42:21 +0000 (10:42 -0800)
committerGuenter Roeck <guenter.roeck@ericsson.com>
Sat, 8 Jan 2011 18:55:46 +0000 (10:55 -0800)
This patch fixes all checkpatch errors and most of the checkpatch warnings.
It also fixes the range check in applesmc_store_fan_speed().

Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
Acked-by: Henrik Rydberg <rydberg@euromail.se>
drivers/hwmon/applesmc.c

index 81792ca471759ad0c373fb9279cd6e3e106dcb36..ce0372f0615ebd37474a1ab9e93ecc3a3d40e79f 100644 (file)
@@ -174,9 +174,8 @@ static int __wait_status(u8 val)
 
        for (us = APPLESMC_MIN_WAIT; us < APPLESMC_MAX_WAIT; us <<= 1) {
                udelay(us);
-               if ((inb(APPLESMC_CMD_PORT) & APPLESMC_STATUS_MASK) == val) {
+               if ((inb(APPLESMC_CMD_PORT) & APPLESMC_STATUS_MASK) == val)
                        return 0;
-               }
        }
 
        return -EIO;
@@ -431,7 +430,7 @@ static int applesmc_has_key(const char *key, bool *value)
 /*
  * applesmc_read_motion_sensor - Read motion sensor (X, Y or Z).
  */
-static int applesmc_read_motion_sensor(int index, s16value)
+static int applesmc_read_motion_sensor(int index, s16 *value)
 {
        u8 buffer[2];
        int ret;
@@ -779,14 +778,12 @@ static ssize_t applesmc_store_fan_speed(struct device *dev,
                                        const char *sysfsbuf, size_t count)
 {
        int ret;
-       u32 speed;
+       unsigned long speed;
        char newkey[5];
        u8 buffer[2];
 
-       speed = simple_strtoul(sysfsbuf, NULL, 10);
-
-       if (speed > 0x4000) /* Bigger than a 14-bit value */
-               return -EINVAL;
+       if (strict_strtoul(sysfsbuf, 10, &speed) < 0 || speed >= 0x4000)
+               return -EINVAL;         /* Bigger than a 14-bit value */
 
        sprintf(newkey, fan_speed_fmt[to_option(attr)], to_index(attr));
 
@@ -822,10 +819,11 @@ static ssize_t applesmc_store_fan_manual(struct device *dev,
 {
        int ret;
        u8 buffer[2];
-       u32 input;
+       unsigned long input;
        u16 val;
 
-       input = simple_strtoul(sysfsbuf, NULL, 10);
+       if (strict_strtoul(sysfsbuf, 10, &input) < 0)
+               return -EINVAL;
 
        ret = applesmc_read_key(FANS_MANUAL, buffer, 2);
        val = (buffer[0] << 8 | buffer[1]);
@@ -1198,24 +1196,24 @@ static __initdata struct dmi_system_id applesmc_whitelist[] = {
          DMI_MATCH(DMI_PRODUCT_NAME, "MacBookAir") },
        },
        { applesmc_dmi_match, "Apple MacBook Pro", {
-         DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-         DMI_MATCH(DMI_PRODUCT_NAME,"MacBookPro") },
+         DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
+         DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro") },
        },
        { applesmc_dmi_match, "Apple MacBook", {
-         DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-         DMI_MATCH(DMI_PRODUCT_NAME,"MacBook") },
+         DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
+         DMI_MATCH(DMI_PRODUCT_NAME, "MacBook") },
        },
        { applesmc_dmi_match, "Apple Macmini", {
-         DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-         DMI_MATCH(DMI_PRODUCT_NAME,"Macmini") },
+         DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
+         DMI_MATCH(DMI_PRODUCT_NAME, "Macmini") },
        },
        { applesmc_dmi_match, "Apple MacPro", {
          DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
          DMI_MATCH(DMI_PRODUCT_NAME, "MacPro") },
        },
        { applesmc_dmi_match, "Apple iMac", {
-         DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-         DMI_MATCH(DMI_PRODUCT_NAME,"iMac") },
+         DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
+         DMI_MATCH(DMI_PRODUCT_NAME, "iMac") },
        },
        { .ident = NULL }
 };