Skip to content

Commit ae28532

Browse files
committed
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 <[email protected]>
1 parent 4cf1aab commit ae28532

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

drivers/hwmon/spd5118.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,8 @@ static int spd5118_i2c_init(struct i2c_client *client)
621621
* If the device type registers return 0, it is possible that the chip
622622
* has a non-zero page selected and takes the specification literally,
623623
* i.e. disables access to volatile registers besides the page register
624-
* if the page is not 0. Try to identify such chips.
624+
* if the page is not 0. The Renesas/ITD SPD5118 Hub Controller is known
625+
* to show this behavior. Try to identify such chips.
625626
*/
626627
if (!regval) {
627628
/* Vendor ID registers must also be 0 */

0 commit comments

Comments
 (0)