hwmon: (lm90) Make SA56004 detection more robust

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 <[email protected]>
Cc: Stijn Devriendt <[email protected]>
Cc: Guenter Roeck <[email protected]>
Acked-by: Stijn Devriendt <[email protected]>
Signed-off-by: Guenter Roeck <[email protected]>

diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c
index 7fef2a1194d9ae6acc088ffd1769138090edce45..7c6a3df36d867bd9256c7fc8fed2b684cf19c207 100644 (file)
--- a/drivers/hwmon/lm90.c
+++ b/drivers/hwmon/lm90.c
@@ -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";
                }
        }