On Thu, 7 Jul 2011 07:11:02 -0700, Guenter Roeck wrote:
> The LM95241 driver accepts every chip ID equal to or larger than 0xA4 as its
> own, and other chips such as LM95245 use chip IDs in the accepted ID range.
> This results in false chip detection.
>
> Fix problem by accepting only the known LM95241 chip ID.
>
> Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
Acked-by: Jean Delvare <khali@xxxxxxxxxxxx>
> ---
> Candidate for -stable.
Definitely.
>
> drivers/hwmon/lm95241.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/hwmon/lm95241.c b/drivers/hwmon/lm95241.c
> index 1a6dfb6..01c638e 100644
> --- a/drivers/hwmon/lm95241.c
> +++ b/drivers/hwmon/lm95241.c
> @@ -339,7 +339,7 @@ static int lm95241_detect(struct i2c_client *new_client,
> if ((i2c_smbus_read_byte_data(new_client, LM95241_REG_R_MAN_ID)
> == MANUFACTURER_ID)
> && (i2c_smbus_read_byte_data(new_client, LM95241_REG_R_CHIP_ID)
> - >= DEFAULT_REVISION)) {
> + == DEFAULT_REVISION)) {
> name = DEVNAME;
> } else {
> dev_dbg(&adapter->dev, "LM95241 detection failed at 0x%02x\n",
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
