There is a reversed condition so the adm1177_write_alert_thr() function
doesn't update "st->alert_threshold_ua".
Fixes: 829091f9c56c ("hwmon: (adm1177) Add ADM1177 Hot Swap Controller and Digital Power Monitor driver")
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
---
drivers/hwmon/adm1177.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hwmon/adm1177.c b/drivers/hwmon/adm1177.c
index 1f2ff7a29306..d314223a404a 100644
--- a/drivers/hwmon/adm1177.c
+++ b/drivers/hwmon/adm1177.c
@@ -63,7 +63,7 @@ static int adm1177_write_alert_thr(struct adm1177_state *st,
ret = i2c_smbus_write_byte_data(st->client, ADM1177_REG_ALERT_TH,
val);
- if (!ret)
+ if (ret)
return ret;
st->alert_threshold_ua = alert_threshold_ua;
--
2.11.0
