Re: [hwmon-next PATCH v5 1/4] dt-bindings: hwmon: add adi,adt7470
Krzysztof Kozlowski <[email protected]> Sat, 1 Aug 2026 18:20:57 +0200
| Newsgroups | org.kernel.vger.linux-pwm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-hwmon,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <20260801-tentacled-nickel-herring-bfabbb@quoll> |
On Thu, Jul 30, 2026 at 06:29:56PM -0300, Luiz Angelo Daros de Luca wrote: > The Analog Devices ADT7470 is a multichannel temperature sensor and > PWM fan controller. It supports monitoring up to 10 external > temperature sensors and controlling up to 4 fans. > > Add the device tree binding documentation for it. This includes > support for the thermal framework by defining the "#thermal-sensor-cells" > property, and models the fan control lines as PWM channels by > defining the "#pwm-cells" property. > > Signed-off-by: Luiz Angelo Daros de Luca <[email protected]> > --- > .../devicetree/bindings/hwmon/adi,adt7470.yaml | 58 ++++++++++++++++++++++ > 1 file changed, 58 insertions(+) > > diff --git a/Documentation/devicetree/bindings/hwmon/adi,adt7470.yaml b/Documentation/devicetree/bindings/hwmon/adi,adt7470.yaml > new file mode 100644 > index 000000000000..f653a06f241a > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/adi,adt7470.yaml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/hwmon/adi,adt7470.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog Devices ADT7470 hwmon sensor > + > +maintainers: > + - Luiz Angelo Daros de Luca <[email protected]> > + > +description: > + Multi-channel temperature monitor and PWM fan controller. > + It supports monitoring up to 10 external temperature sensors and > + controlling up to 4 fans. The fan control lines are modeled > + as standard PWM channels. > + > +allOf: > + - if: > + required: > + - "#thermal-sensor-cells" > + then: > + $ref: /schemas/thermal/thermal-sensor.yaml# Place the entire "allOf:" after "required:" block. Reviewed-by: Krzysztof Kozlowski <[email protected]> Best regards, Krzysztof