kevin/linux
1
Fork 0
Code

hwmon: (max6697) Document discrepancy in overtemperature status bit values

Browse Source 
In the MAX6581 datasheet Revision 0 to 3, the local channel overtemperature
status is reported in bit 6 of register 0x45, and the overtemperature
status for remote channel 7 is reported in bit 7. In Revision 4 and later,
the local channel overtemperature status is reported in bit 7, and the
remote channel 7 overtemperature status is reported in bit 6. A real
chip was found to match the functionality documented in Revision 4 and
later.

The code was fixed with commit 1ea3fd1eb9 ("hwmon: (max6697) Fix swapped
temp{1,8} critical alarms"). At that time it looked like this was an
original bug. It only turned out later that the problem was the result of
incorrect information in the chip datasheet.

Document the discrepancy to avoid confusion caused by old versions of the
datasheet.

Cc: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Tzung-Bi Shih <tzungbi@kernel.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
This commit is contained in:
Guenter Roeck 2024-07-27 12:06:04 -07:00
parent c1b93b5414
commit 3adc2857bb
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
drivers/hwmon/max6697.c
View File

@ -248,6 +248,16 @@ static int max6697_read(struct device *dev, enum hwmon_sensor_types type,
ret = regmap_read(regmap, MAX6697_REG_STAT_CRIT, &regval);
if (ret)
return ret;
/*
* In the MAX6581 datasheet revision 0 to 3, the local channel
* overtemperature status is reported in bit 6 of register 0x45,
* and the overtemperature status for remote channel 7 is
* reported in bit 7. In Revision 4 and later, the local channel
* overtemperature status is reported in bit 7, and the remote
* channel 7 overtemperature status is reported in bit 6. A real
* chip was found to match the functionality documented in
* Revision 4 and later.
*/
*val = !!(regval & BIT(channel ? channel - 1 : 7));
break;
case hwmon_temp_max_alarm: