Re: [PATCH v3 1/2] dt-bindings: hwmon: moortec,mr75203: adapt multipleOf for T-Head TH1520
From: Drew Fustini <fustini@kernel.org>
Date: 2026-03-16 14:22:01
Also in:
linux-hwmon, linux-riscv, lkml
On Mon, Mar 16, 2026 at 07:11:37AM -0700, Guenter Roeck wrote:
On 3/15/26 05:12, Conor Dooley wrote:quoted
On Sat, Mar 14, 2026 at 09:56:27AM -0700, Drew Fustini wrote:quoted
On Tue, Mar 10, 2026 at 12:24:56AM +0800, Icenowy Zheng wrote:quoted
The G and J coefficients provided by T-Head TH1520 manual (which calls them A and C coefficients and calls H coefficient in the binding as B) have 1/100 degree Celsius precision (the values are 42.74 and -0.16 respectively), however the binding currently only allows coefficients as precise as 100 milli-Celsius (1/10 degree Celsius). Change the multipleOf value of these two coefficients to 10 (in the unit of milli-Celsius) to satisfy the need of TH1520. Signed-off-by: Icenowy Zheng <zhengxingda@iscas.ac.cn> Reviewed-by: Drew Fustini <fustini@kernel.org> --- Changes in v3: - Added Drew's R-b. - Fixed some typos in the commit message and slightly reworded the precision sentence. Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)diff --git a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml index 56db2292f062d..7d57c2934a8a1 100644 --- a/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml +++ b/Documentation/devicetree/bindings/hwmon/moortec,mr75203.yaml@@ -105,7 +105,7 @@ properties: G coefficient for temperature equation. Default for series 5 = 60000 Default for series 6 = 57400 - multipleOf: 100 + multipleOf: 10 minimum: 1000 $ref: /schemas/types.yaml#/definitions/uint32@@ -131,7 +131,7 @@ properties: J coefficient for temperature equation. Default for series 5 = -100 Default for series 6 = 0 - multipleOf: 100 + multipleOf: 10 maximum: 0 $ref: /schemas/types.yaml#/definitions/int32-- 2.52.0I have applied this to thead-dt-for-next.That's odd, why is a hwmon binding in your branch?I don't see an ack on it from the relevant maintainer either?I don't get it either. Normally I am the one to apply hwmon bindings through the hwmon branch.
Sorry, I should not have done this. I should only be applying dts/dtsi patches to my thead dt branch. Thanks, Drew