Re: [PATCH 1/1] dt-bindings: hwmon: tmp102: move ti,tmp103 out of trivial-devices.yaml

[Guenter Roeck]

On 8/30/26 02:53, Krzysztof Kozlowski wrote:
> On Wed, Aug 26, 2026 at 05:14:19PM -0400, Frank.Li@xxxxxxxxxxx wrote:
> > From: Frank Li <Frank.Li@xxxxxxx>
> >
> > Move ti,tmp103 binding from trivial-devices.yaml to ti,tmp102.yaml. Both
> > devices are single temperature sensors and update "#thermal-sensor-cells"
> > property to accept values 0 and 1 (passing no argument is equivalent to
> > passing 0 as the first argument)
>
> Why updating the property? If there are single sensors, then what is the
> cell for?

It registers as temperature sensor with the thermal subsystem. The documentation
of #thermal-sensor-cells states that the property is mandatory and "... Will be
0 on sensor nodes with only a single sensor". I don't claim to understand how
this is supposed to work, but the thermal subsystem code looking for it does
seem to suggest that the property is mandatory, even if there is only a single
sensor.

Thanks,
Guenter

> > Fix below CHECK_DTBS warnings:
> > arch/arm/boot/dts/nxp/imx/imx6dl-plym2m.dtb: temperature-sensor@70 (ti,tmp103): '#thermal-sensor-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
> >
> > Signed-off-by: Frank Li <Frank.Li@xxxxxxx>
> > ---
> >   Documentation/devicetree/bindings/hwmon/ti,tmp102.yaml | 5 +++--
> >   Documentation/devicetree/bindings/trivial-devices.yaml | 2 --
> >   2 files changed, 3 insertions(+), 4 deletions(-)
>
> Best regards,
> Krzysztof