diff mbox series

[v2] hwmon: (ad7314) Validate leading zero bits and return error

Message ID 24a50c2981a318580aca8f50d23be7987b69ea00.camel@iris-sensing.com (mailing list archive)
State Accepted
Headers show
Series [v2] hwmon: (ad7314) Validate leading zero bits and return error | expand

Commit Message

Erik Schumacher Feb. 24, 2025, 9:19 a.m. UTC
Leading zero bits are sent on the bus before the temperature value is
transmitted. If any of these bits are high, the connection might be
unstable or there could be no AD7314 / ADT730x (or compatible) at all.
Return -EIO in that case.

Signed-off-by: Erik Schumacher <erik.schumacher@iris-sensing.com>
---

Changes v1 -> v2:
- return EIO instead of ENXIO
- check leading zeros for ADT7301/2 as well

 drivers/hwmon/ad7314.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Guenter Roeck March 2, 2025, 4:41 p.m. UTC | #1
On Mon, Feb 24, 2025 at 09:19:04AM +0000, Erik Schumacher wrote:
> Leading zero bits are sent on the bus before the temperature value is
> transmitted. If any of these bits are high, the connection might be
> unstable or there could be no AD7314 / ADT730x (or compatible) at all.
> Return -EIO in that case.
> 
> Signed-off-by: Erik Schumacher <erik.schumacher@iris-sensing.com>

Applied.

Thanks,
Guenter
diff mbox series

Patch

diff --git a/drivers/hwmon/ad7314.c b/drivers/hwmon/ad7314.c
index 7802bbf5f958..59424103f634 100644
--- a/drivers/hwmon/ad7314.c
+++ b/drivers/hwmon/ad7314.c
@@ -22,11 +22,13 @@ 
  */
 #define AD7314_TEMP_MASK		0x7FE0
 #define AD7314_TEMP_SHIFT		5
+#define AD7314_LEADING_ZEROS_MASK	BIT(15)
 
 /*
  * ADT7301 and ADT7302 temperature masks
  */
 #define ADT7301_TEMP_MASK		0x3FFF
+#define ADT7301_LEADING_ZEROS_MASK	(BIT(15) | BIT(14))
 
 enum ad7314_variant {
 	adt7301,
@@ -65,12 +67,20 @@  static ssize_t ad7314_temperature_show(struct device *dev,
 		return ret;
 	switch (spi_get_device_id(chip->spi_dev)->driver_data) {
 	case ad7314:
+		if (ret & AD7314_LEADING_ZEROS_MASK) {
+			/* Invalid read-out, leading zero part is missing */
+			return -EIO;
+		}
 		data = (ret & AD7314_TEMP_MASK) >> AD7314_TEMP_SHIFT;
 		data = sign_extend32(data, 9);
 
 		return sprintf(buf, "%d\n", 250 * data);
 	case adt7301:
 	case adt7302:
+		if (ret & ADT7301_LEADING_ZEROS_MASK) {
+			/* Invalid read-out, leading zero part is missing */
+			return -EIO;
+		}
 		/*
 		 * Documented as a 13 bit twos complement register
 		 * with a sign bit - which is a 14 bit 2's complement