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

Uwe Kleine-König <[email protected]> Thu, 23 Jul 2026 08:10:07 +0200

Newsgroups	org.kernel.vger.linux-pwm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-hwmon,org.kernel.vger.linux-kernel
Message-ID	<amGvR3VwfDN1Gi0Q@monoceros>

Hi Luiz,

On Mon, Jul 20, 2026 at 11:12:49PM -0300, Luiz Angelo Daros de Luca wrote:
> Initially, I modeled the driver purely as a cooling device. However,
> following Sashiko-dev's suggestion in a previous review, I converted
> it to expose PWM channels so it could be used with the generic pwm-fan
> driver (which indeed makes the code cleaner). My primary experience is
> with DSA switches, so hwmon is a new area for me.

I like the driver exposing a generic PWM. For me it's quite in line with
the "Linux spirit" to do things with devices that are possible even if
that doesn't figure in the vendor's scheme.

> Whatever you and the DT maintainers agree on, I will gladly adjust for v3.

But my opinion doesn't count ... :-)

Best regards
Uwe

signature.asc (application/pgp-signature, 488 B)