[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241216173648.526-1-m.masimov@maxima.ru>
Date: Mon, 16 Dec 2024 20:36:45 +0300
From: Murad Masimov <m.masimov@...ima.ru>
To: Eric Tremblay <etremblay@...tech-controls.com>
CC: Jean Delvare <jdelvare@...e.com>, Guenter Roeck <linux@...ck-us.net>,
<linux-hwmon@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<lvc-project@...uxtesting.org>, Murad Masimov <m.masimov@...ima.ru>
Subject: [PATCH 0/3] hwmon: (tmp513) Fix interpretation of values of TMP513 registers
The function tmp51x_get_value returns processed values of the TMP513 device
registers. Raw register values are converted to signed integer values by sign
extension in accordance with the algorithm provided in the specification, but
due to the off-by-one error in the sign bit index, the result is incorrect.
There are also some other mistakes, such as incorrect cast.
Changes introduced by these patches are based on the TMP512/TMP513 datasheets
that are specified in Documentation/hwmon/tmp513.rst. They have not actually
been tested in any real or virtual environment. However the calculations have
been tested separately to make sure they work as expected.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Murad Masimov (3):
hwmon: (tmp513) Fix interpretation of values of Shunt Voltage and
Limit Registers
hwmon: (tmp513) Fix Current Register value interpretation
hwmon: (tmp513) Fix interpretation of values of Temperature Result and
Limit Registers
drivers/hwmon/tmp513.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
--
2.39.2
Powered by blists - more mailing lists