[PATCH v3 1/2] dt-bindings: hwmon: moortec,mr75203: adapt multipleOf for T-Head TH1520
Drew Fustini
fustini at kernel.org
Sun Mar 15 08:44:02 PDT 2026
On Sun, Mar 15, 2026 at 12:12:13PM +0000, Conor Dooley wrote:
> On Sat, Mar 14, 2026 at 09:56:27AM -0700, Drew Fustini wrote:
> > On Tue, Mar 10, 2026 at 12:24:56AM +0800, Icenowy Zheng wrote:
> > > 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 at iscas.ac.cn>
> > > Reviewed-by: Drew Fustini <fustini at 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.0
> > >
> >
> > I have applied this to thead-dt-for-next.
>
> That's odd, why is a hwmon binding in your branch?
>
> > https://git.kernel.org/pub/scm/linux/kernel/git/fustini/linux.git/commit/?h=thead-dt-for-next&id=eb4fd43ff6793681f0a0a8e8ac861142caba2b19
>
> I don't see an ack on it from the relevant maintainer either?
Sorry, my mistake, I should not have applied the binding. And you are
right to point out that there was no ack from Guenter or Rahul Tanwar.
I will drop both patches from thead-dt-for-next and wait for agreement
on the binding.
Thanks,
Drew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20260315/5c1aac5d/attachment.sig>
More information about the linux-riscv
mailing list