| Message ID | 20240322-hwmon_dtschema-v2-2-570bee1acecb@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | dt-bindings: hwmon: convert multiple devices to dtschema | expand |
On Fri, Mar 22, 2024 at 07:45:27AM +0100, Javier Carrasco wrote: > Convert existing binding to support validation. > > This is a straightforward conversion with now new properties. > > Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> > --- > .../devicetree/bindings/hwmon/ibm,powernv.yaml | 37 ++++++++++++++++++++++ > .../devicetree/bindings/hwmon/ibmpowernv.txt | 23 -------------- > 2 files changed, 37 insertions(+), 23 deletions(-) > > diff --git a/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml b/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml > new file mode 100644 > index 000000000000..b26d42bce938 > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml Filename should really match one of the compatibles. "powernv" is not a type of hwmon as far as a quick google reports so I think its a poor choice here regardless. With a compatible for the filename: Reviewed-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor. > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/hwmon/ibm,powernv.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: IBM POWERNV platform sensors > + > +maintainers: > + - Javier Carrasco <javier.carrasco.cruz@gmail.com> > + > +properties: > + compatible: > + enum: > + - ibm,opal-sensor-cooling-fan > + - ibm,opal-sensor-amb-temp > + - ibm,opal-sensor-power-supply > + - ibm,opal-sensor-power > + > + sensor-id: > + description: > + An opaque id provided by the firmware to the kernel, identifies a > + given sensor and its attribute data. > + $ref: /schemas/types.yaml#/definitions/uint32 > + > +required: > + - compatible > + - sensor-id > + > +additionalProperties: false > + > +examples: > + - | > + sensor { > + compatible = "ibm,opal-sensor-cooling-fan"; > + sensor-id = <0x7052107>; > + }; > diff --git a/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt b/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt > deleted file mode 100644 > index f93242be60a1..000000000000 > --- a/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt > +++ /dev/null > @@ -1,23 +0,0 @@ > -IBM POWERNV platform sensors > ----------------------------- > - > -Required node properties: > -- compatible: must be one of > - "ibm,opal-sensor-cooling-fan" > - "ibm,opal-sensor-amb-temp" > - "ibm,opal-sensor-power-supply" > - "ibm,opal-sensor-power" > -- sensor-id: an opaque id provided by the firmware to the kernel, identifies a > - given sensor and its attribute data > - > -Example sensors node: > - > -cooling-fan#8-data { > - sensor-id = <0x7052107>; > - compatible = "ibm,opal-sensor-cooling-fan"; > -}; > - > -amb-temp#1-thrs { > - sensor-id = <0x5096000>; > - compatible = "ibm,opal-sensor-amb-temp"; > -}; > > -- > 2.40.1 >
diff --git a/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml b/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml new file mode 100644 index 000000000000..b26d42bce938 --- /dev/null +++ b/Documentation/devicetree/bindings/hwmon/ibm,powernv.yaml @@ -0,0 +1,37 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/hwmon/ibm,powernv.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: IBM POWERNV platform sensors + +maintainers: + - Javier Carrasco <javier.carrasco.cruz@gmail.com> + +properties: + compatible: + enum: + - ibm,opal-sensor-cooling-fan + - ibm,opal-sensor-amb-temp + - ibm,opal-sensor-power-supply + - ibm,opal-sensor-power + + sensor-id: + description: + An opaque id provided by the firmware to the kernel, identifies a + given sensor and its attribute data. + $ref: /schemas/types.yaml#/definitions/uint32 + +required: + - compatible + - sensor-id + +additionalProperties: false + +examples: + - | + sensor { + compatible = "ibm,opal-sensor-cooling-fan"; + sensor-id = <0x7052107>; + }; diff --git a/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt b/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt deleted file mode 100644 index f93242be60a1..000000000000 --- a/Documentation/devicetree/bindings/hwmon/ibmpowernv.txt +++ /dev/null @@ -1,23 +0,0 @@ -IBM POWERNV platform sensors ----------------------------- - -Required node properties: -- compatible: must be one of - "ibm,opal-sensor-cooling-fan" - "ibm,opal-sensor-amb-temp" - "ibm,opal-sensor-power-supply" - "ibm,opal-sensor-power" -- sensor-id: an opaque id provided by the firmware to the kernel, identifies a - given sensor and its attribute data - -Example sensors node: - -cooling-fan#8-data { - sensor-id = <0x7052107>; - compatible = "ibm,opal-sensor-cooling-fan"; -}; - -amb-temp#1-thrs { - sensor-id = <0x5096000>; - compatible = "ibm,opal-sensor-amb-temp"; -};
Convert existing binding to support validation. This is a straightforward conversion with now new properties. Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com> --- .../devicetree/bindings/hwmon/ibm,powernv.yaml | 37 ++++++++++++++++++++++ .../devicetree/bindings/hwmon/ibmpowernv.txt | 23 -------------- 2 files changed, 37 insertions(+), 23 deletions(-)