[PATCH RFC 01/16] dt-bindings: hwmon: Convert aspeed,ast2400-pwm-tacho to DT schema
Rob Herring
robh at kernel.org
Thu Dec 18 02:37:28 AEDT 2025
On Thu, Dec 11, 2025 at 2:46 AM Andrew Jeffery
<andrew at codeconstruct.com.au> wrote:
>
> From: "Rob Herring (Arm)" <robh at kernel.org>
>
> Convert the ASpeed fan controller binding to DT schema format.
>
> The '#cooling-cells' value used is 1 rather than 2. '#size-cells' is 0
> rather 1.
Okay, I can't figure out why I thought '#cooling-cells' needed to be 1
here. I don't see that anywhere in the tree. The driver for sure only
supports 2, so anything that's not is an error in any case.
> Some users define more that 8 fan nodes where 2 fans share a PWM. The
> driver seems to let the 2nd fan just overwrite the 1st one. That also
> creates some addressing errors in the DT (duplicate addresses and wrong
> unit-addresses).
>
> Signed-off-by: Rob Herring (Arm) <robh at kernel.org>
> Signed-off-by: Andrew Jeffery <andrew at codeconstruct.com.au>
> ---
> .../bindings/hwmon/aspeed,ast2400-pwm-tacho.yaml | 106 +++++++++++++++++++++
> .../devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 73 --------------
> 2 files changed, 106 insertions(+), 73 deletions(-)
More information about the Linux-aspeed
mailing list