Re: [PATCH v3 2/4] hwmon: (ina238) Correctly clamp temperature
From: Guenter Roeck <linux@roeck-us.net>
Date: 2025-08-29 09:55:07
Also in:
linux-doc, linux-hwmon, lkml
On 8/28/25 20:05, Chris Packham wrote:
quoted hunk ↗ jump to hunk
ina238_write_temp() was attempting to clamp the user input but was throwing away the result. Ensure that we clamp the value to the appropriate range before it is converted into a register value. Fixes: 0d9f596b1fe3 ("hwmon: (ina238) Modify the calculation formula to adapt to different chips") Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> --- Notes: Changes in v3: - New. Split off bugfix from main patch drivers/hwmon/ina238.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/hwmon/ina238.c b/drivers/hwmon/ina238.c index 5a394eeff676..4d3dc018ead9 100644 --- a/drivers/hwmon/ina238.c +++ b/drivers/hwmon/ina238.c@@ -572,7 +572,7 @@ static int ina238_write_temp(struct device *dev, u32 attr, long val) return -EOPNOTSUPP; /* Signed */ - regval = clamp_val(val, -40000, 125000); + val = clamp_val(val, -40000, 125000);
That needs another correction: As it turns out, the default register value is 0x7ff0, or 255875. That means we need to accept that range. The same is probably true for negative temperatures, but I'll need to see the real chip to be sure. Yes, the chips only support a limited temperature range, but that is the limit register, not the supported range. Other chips have a similar problem. It is ok to limit the input range if the chip has a reasonable default set, but if the actual chip default is 0x7ff0 or 255.875 degrees C we need to support writing that value. Guenter