Re: [PATCH 1/4] dt-bindings: hwmon: add binding for adi,adt7470

From: Rob Herring

Date: Wed Jul 22 2026 - 13:56:32 EST


On Thu, Jul 16, 2026 at 11:33:53PM -0300, Luiz Angelo Daros de Luca wrote:
> > + "#cooling-cells":
> > + const: 2
> > + description: |
> > + Cooling device exposing the 4 PWM channels.
>
> Since "thermal/of: Support cooling device ID in cooling-spec", I could
> be 2 or 3. I don't know if I should use enum [2, 3] or just go
> directly to the new const: 3.
> My test device has only the 0 channel in use and 2 is fine for now.
>
> Sashiko also asked to require #pwm-cells, but it would mean to
> implement a parser for that too. I don't know if it is worth it but I
> can do it if needed.

What does implement a parser mean? You can define the property now, but
support it in a driver later.

Rob