]> www.infradead.org Git - users/jedix/linux-maple.git/commit
hwmon: (max6697) Document discrepancy in overtemperature status bit values
authorGuenter Roeck <linux@roeck-us.net>
Sat, 27 Jul 2024 19:06:04 +0000 (12:06 -0700)
committerGuenter Roeck <linux@roeck-us.net>
Wed, 31 Jul 2024 17:43:53 +0000 (10:43 -0700)
commit3adc2857bb6a7c6cb669627b87e9998a3a28231d
treeb77ddbab9c8f7f3136e4a145a9ed4df89928724d
parentc1b93b5414b48054600c96e28c325ad5920a47e6
hwmon: (max6697) Document discrepancy in overtemperature status bit values

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 1ea3fd1eb986 ("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>
drivers/hwmon/max6697.c