diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2024-11-17 07:55:32 -0800 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2025-04-23 07:18:27 -0700 |
| commit | ae28532aff1f7912c1d118b4257c095ce62f1cb0 (patch) | |
| tree | db5da5768e1eca09fe41e8c3986391e9c9544040 | |
| parent | 4cf1aab45cc56e9276924c21b79da42e2eac01df (diff) | |
hwmon: (spd5118) Name chips taking the specification literally
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>
| -rw-r--r-- | drivers/hwmon/spd5118.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/spd5118.c b/drivers/hwmon/spd5118.c index 02eb21684c3a..c5ad06b90cbf 100644 --- a/drivers/hwmon/spd5118.c +++ b/drivers/hwmon/spd5118.c @@ -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 */ |