]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
hwmon: (spd5118) Name chips taking the specification literally
authorGuenter Roeck <linux@roeck-us.net>
Sun, 17 Nov 2024 15:55:32 +0000 (07:55 -0800)
committerGuenter Roeck <linux@roeck-us.net>
Wed, 23 Apr 2025 14:18:27 +0000 (07:18 -0700)
The Renesas/IDT SPD5118 Hub Controller is known to take the specification
literally and does not permit access to volatile registers except for the
page register if the selected page is non-zero. Explicitly name the chip
to ensure that the information does not get lost.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/spd5118.c

index 02eb21684c3a6614449f481291201162bf9c64c9..c5ad06b90cbf429dd38bb54e20f3cc9ca3f47e90 100644 (file)
@@ -621,7 +621,8 @@ static int spd5118_i2c_init(struct i2c_client *client)
         * If the device type registers return 0, it is possible that the chip
         * has a non-zero page selected and takes the specification literally,
         * i.e. disables access to volatile registers besides the page register
-        * if the page is not 0. Try to identify such chips.
+        * if the page is not 0. The Renesas/ITD SPD5118 Hub Controller is known
+        * to show this behavior. Try to identify such chips.
         */
        if (!regval) {
                /* Vendor ID registers must also be 0 */