From: Guenter Roeck <linux@roeck-us.net>
Date: Sun, 28 Jul 2024 14:09:54 +0000 (-0700)
Subject: hwmon: (max1619) Mask valid alarm bits
X-Git-Tag: v6.12-rc1~150^2~58
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=02c3c4e7026b307a220132b59758432b6e7a0cac;p=thirdparty%2Flinux.git

hwmon: (max1619) Mask valid alarm bits

Bit 0, 5, and 6 in the status register are reserved and, if set, do not
indicate an alarm. Bit 7 is the 'busy' bit and also does not indicate
an alarm. Mask the non-alarm bits to avoid reporting them to userspace.

Cc: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Tzung-Bi Shih <tzungbi@kernel.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/max1619.c b/drivers/hwmon/max1619.c
index 8eb7d04bd2f5e..5edc9bbbe2995 100644
--- a/drivers/hwmon/max1619.c
+++ b/drivers/hwmon/max1619.c
@@ -112,6 +112,7 @@ static struct max1619_data *max1619_update_device(struct device *dev)
 		config = i2c_smbus_read_byte_data(client, MAX1619_REG_R_CONFIG);
 		if (!(config & 0x20))
 			data->alarms ^= 0x02;
+		data->alarms &= 0x1e;
 
 		data->last_updated = jiffies;
 		data->valid = true;