From: Conor Dooley <conor@kernel.org> To: Javier Carrasco <javier.carrasco.cruz@gmail.com> Cc: Jean Delvare <jdelvare@suse.com>, Guenter Roeck <linux@roeck-us.net>, Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Michael Ellerman <mpe@ellerman.id.au>, Nicholas Piggin <npiggin@gmail.com>, Christophe Leroy <christophe.leroy@csgroup.eu>, "Aneesh Kumar K.V" <aneesh.kumar@kernel.org>, "Naveen N. Rao" <naveen.n.rao@linux.ibm.com>, linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH v2 2/5] dt-bindings: hwmon: ibmpowernv: convert to dtschema Date: Fri, 22 Mar 2024 17:59:37 +0000 [thread overview] Message-ID: <20240322-papaya-diabolic-e7c2c4319eb2@spud> (raw) In-Reply-To: <20240322-hwmon_dtschema-v2-2-570bee1acecb@gmail.com> [-- Attachment #1: Type: text/plain, Size: 3029 bytes --] 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 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor@kernel.org> To: Javier Carrasco <javier.carrasco.cruz@gmail.com> Cc: linux-hwmon@vger.kernel.org, Rob Herring <robh@kernel.org>, Conor Dooley <conor+dt@kernel.org>, Jean Delvare <jdelvare@suse.com>, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Nicholas Piggin <npiggin@gmail.com>, "Aneesh Kumar K.V" <aneesh.kumar@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, "Naveen N. Rao" <naveen.n.rao@linux.ibm.com>, linuxppc-dev@lists.ozlabs.org, Guenter Roeck <linux@roeck-us.net> Subject: Re: [PATCH v2 2/5] dt-bindings: hwmon: ibmpowernv: convert to dtschema Date: Fri, 22 Mar 2024 17:59:37 +0000 [thread overview] Message-ID: <20240322-papaya-diabolic-e7c2c4319eb2@spud> (raw) In-Reply-To: <20240322-hwmon_dtschema-v2-2-570bee1acecb@gmail.com> [-- Attachment #1: Type: text/plain, Size: 3029 bytes --] 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 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2024-03-22 17:59 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-22 6:45 [PATCH v2 0/5] dt-bindings: hwmon: convert multiple devices to dtschema Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco 2024-03-22 6:45 ` [PATCH v2 1/5] dt-bindings: hwmon: as370: convert " Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco 2024-03-22 6:45 ` [PATCH v2 2/5] dt-bindings: hwmon: ibmpowernv: " Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco 2024-03-22 17:59 ` Conor Dooley [this message] 2024-03-22 17:59 ` Conor Dooley 2024-03-22 6:45 ` [PATCH v2 3/5] dt-bindings: hwmon: pwm-fan: drop text file Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco 2024-03-22 6:45 ` [PATCH v2 4/5] dt-bindings: hwmon: stts751: convert to dtschema Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco 2024-03-22 18:00 ` Conor Dooley 2024-03-22 18:00 ` Conor Dooley 2024-03-22 6:45 ` [PATCH v2 5/5] dt-bindings: hwmon: ibm,p8-occ-hwmon: move to trivial devices Javier Carrasco 2024-03-22 6:45 ` Javier Carrasco
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20240322-papaya-diabolic-e7c2c4319eb2@spud \ --to=conor@kernel.org \ --cc=aneesh.kumar@kernel.org \ --cc=christophe.leroy@csgroup.eu \ --cc=conor+dt@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=javier.carrasco.cruz@gmail.com \ --cc=jdelvare@suse.com \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-hwmon@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=mpe@ellerman.id.au \ --cc=naveen.n.rao@linux.ibm.com \ --cc=npiggin@gmail.com \ --cc=robh@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.