Re: [PATCH] dt-bindings: trivial-devices: Add National lm77 temperature sensor

["Łukasz Majewski" <lukma@nabladev.com>]

Hi Guenter,

> On 11/10/26 07:02, Lukasz Majewski wrote:
> > Add "national,lm77" as compatible for the National (now TI) LM77
> > digital I2C temperature sensor.
> > 
> > Signed-off-by: Lukasz Majewski <lukma@nabladev.com>
> > ---
> >   Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml
> > b/Documentation/devicetree/bindings/trivial-devices.yaml index
> > a8b2b9deade4..9413ee0626f5 100644 ---
> > a/Documentation/devicetree/bindings/trivial-devices.yaml +++
> > b/Documentation/devicetree/bindings/trivial-devices.yaml @@ -387,6
> > +387,8 @@ properties:
> >             - national,lm85c
> >               # I2C ±0.33°C Accurate, 12-Bit + Sign Temperature
> > Sensor and Thermal Window Comparator
> >             - national,lm92
> > +            # Temperature sensor
> > +          - national,lm77  
> 
> I'd suggest to retain alphabetic order.
> 

The "alphabetic" order for "national," is already "disturbed" in this
file - that is why I've put it in the end.

Now, the question - shall I "order" properly all of "national,"
bindings in a single patch (including the new one)?

Or prepare two patches:
- One to make the alphabetic order for already in-tree properties 

and

- Add the "national,lm77" afterwards ?

> Having said that, is there any evidence that this chip is used
> on a system supporting devicetree ?

Yes, it is required for CBASE board (also send to the devicetree mailing
list for review).

> 
> Thanks,
> Guenter
> 
> >               # Nuvoton Temperature Sensor
> >             - nuvoton,w83773g
> >               # NXP ISP1301 USB transceiver  
> 



-- 
Best regards,

Lukasz Majewski

--
Nabla Software Engineering GmbH
HRB 40522 Augsburg
Phone: +49 821 45592596
E-Mail: office@nabladev.com
Managing Director : Stefano Babic