hwmon: (lm90) Fix usage of CONFIG2 register in detect function
authorGuenter Roeck <linux@roeck-us.net>
Sat, 6 Nov 2021 17:02:44 +0000 (10:02 -0700)
committerGuenter Roeck <linux@roeck-us.net>
Mon, 13 Dec 2021 00:22:53 +0000 (16:22 -0800)
commitfce15c45d3fbd9fc1feaaf3210d8e3f8b33dfd3a
treebbd7fc9a6adf793c7a1c8eaad4dd6c3ecdc8cbc5
parent2585cf9dfaaddf00b069673f27bb3f8530e2039c
hwmon: (lm90) Fix usage of CONFIG2 register in detect function

The detect function had a comment "Make compiler happy" when id did not
read the second configuration register. As it turns out, the code was
checking the contents of this register for manufacturer ID 0xA1 (NXP
Semiconductor/Philips), but never actually read the register. So it
wasn't surprising that the compiler complained, and it indeed had a point.
Fix the code to read the register contents for manufacturer ID 0xa1.

At the same time, the code was reading the register for manufacturer ID
0x41 (Analog Devices), but it was not using the results. In effect it was
just checking if reading the register returned an error. That doesn't
really add much if any value, so stop doing that.

Fixes: f90be42fb383 ("hwmon: (lm90) Refactor reading of config2 register")
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/lm90.c