Commit ae28532a authored by Guenter Roeck's avatar Guenter Roeck
Browse files

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: default avatarGuenter Roeck <linux@roeck-us.net>
parent 4cf1aab4

drivers/hwmon/spd5118.c

+2 −1
Original line number Diff line number Diff line
@@ -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 */