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@xxxxxxxxxxxx>
Cc: Stijn Devriendt <sdevrien@xxxxxxxxx>
Cc: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
---
drivers/hwmon/lm90.c | 15 +++++++++++++--
1 file changed, 13 insertions(+), 2 deletions(-)
--- linux-3.0-rc6.orig/drivers/hwmon/lm90.c 2011-07-07 13:35:10.000000000 +0200
+++ linux-3.0-rc6/drivers/hwmon/lm90.c 2011-07-07 14:26:10.000000000 +0200
@@ -1283,8 +1283,19 @@ static int lm90_detect(struct i2c_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";
}
}
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
