hwmon: (lm90) Make SA56004 detection more robust
authorJean Delvare <khali@linux-fr.org>
Sun, 24 Jul 2011 18:36:15 +0000 (20:36 +0200)
committerGuenter Roeck <guenter.roeck@ericsson.com>
Fri, 29 Jul 2011 03:16:36 +0000 (20:16 -0700)
With a device ID register value of 0, the SA56004 detection is rather
weak. Check several other register too to confirm the detection, as we
do for other supported devices.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: Stijn Devriendt <sdevrien@cisco.com>
Cc: Guenter Roeck <guenter.roeck@ericsson.com>
Acked-by: Stijn Devriendt <sdevrien@cisco.com>
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/lm90.c

index 7fef2a1194d9ae6acc088ffd1769138090edce45..7c6a3df36d867bd9256c7fc8fed2b684cf19c207 100644 (file)
@@ -1283,8 +1283,19 @@ static int lm90_detect(struct i2c_client *new_client,
                        }
                }
        } else
-       if (man_id == 0xA1) { /*  NXP Semiconductor/Philips */
-               if (chip_id == 0x00 && address >= 0x48 && address <= 0x4F) {
+       if (address >= 0x48 && address <= 0x4F
+        && man_id == 0xA1) { /*  NXP Semiconductor/Philips */
+               int reg_config2;
+
+               reg_config2 = i2c_smbus_read_byte_data(new_client,
+                                               LM90_REG_R_CONFIG2);
+               if (reg_config2 < 0)
+                       return -ENODEV;
+
+               if (chip_id == 0x00
+                && (reg_config1 & 0x2A) == 0x00
+                && (reg_config2 & 0xFE) == 0x00
+                && reg_convrate <= 0x09) {
                        name = "sa56004";
                }
        }