From: Guenter Roeck <linux@roeck-us.net>
Date: Sun, 11 Mar 2018 01:49:47 +0000 (-0800)
Subject: hwmon: (pmbus/max8688) Accept negative page register values
X-Git-Tag: v4.17-rc1~66^2~15
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=a46f8cd696624ef757be0311eb28f119c36778e8;p=linux-block.git

hwmon: (pmbus/max8688) Accept negative page register values

A negative page register value means that no page needs to be
selected. This is used by status register evaluations and needs
to be accepted.

Fixes: da8e48ab483e1 ("hwmon: (pmbus) Always call _pmbus_read_byte in core driver")
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/pmbus/max8688.c b/drivers/hwmon/pmbus/max8688.c
index dd4883a19045..e951f9b87abb 100644
--- a/drivers/hwmon/pmbus/max8688.c
+++ b/drivers/hwmon/pmbus/max8688.c
@@ -45,7 +45,7 @@ static int max8688_read_word_data(struct i2c_client *client, int page, int reg)
 {
 	int ret;
 
-	if (page)
+	if (page > 0)
 		return -ENXIO;
 
 	switch (reg) {