* [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 [not found] <20230801163546.3170-1-mail@carsten-spiess.de> @ 2023-08-01 16:35 ` Carsten Spieß 2023-08-01 20:52 ` Conor Dooley 0 siblings, 1 reply; 6+ messages in thread From: Carsten Spieß @ 2023-08-01 16:35 UTC (permalink / raw) To: Carsten Spieß, Jean Delvare, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm Cc: linux-hwmon, devicetree, linux-kernel, linux-renesas-soc Add dt-bindings for Renesas ISL28022 power monitor. Signed-off-by: Carsten Spieß <mail@carsten-spiess.de> --- v3: - changelog added v2/v3: - schema errors fixed - properties reworked - shunt-resistor minimum and default value added --- .../bindings/hwmon/renesas,isl28022.yaml | 65 +++++++++++++++++++ MAINTAINERS | 1 + 2 files changed, 66 insertions(+) create mode 100644 Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml diff --git a/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml b/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml new file mode 100644 index 000000000000..1e0971287941 --- /dev/null +++ b/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml @@ -0,0 +1,65 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/hwmon/renesas,isl28022.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Renesas ISL28022 power monitor + +maintainers: + - Carsten Spieß <mail@carsten-spiess.de> + +description: | + The ISL28022 is a power monitor with I2C interface. The device monitors + voltage, current via shunt resistor and calculated power. + + Datasheets: + https://www.renesas.com/us/en/www/doc/datasheet/isl28022.pdf + +properties: + compatible: + enum: + - renesas,isl28022 + + reg: + maxItems: 1 + + shunt-resistor-micro-ohms: + description: | + Shunt resistor value in micro-Ohm + minimum: 800 + default: 10000 + + renesas,shunt-range-microvolt: + description: | + Maximal shunt voltage range of +/- 40 mV, 80 mV, 160 mV or 320 mV + default: 320000 + enum: [40000, 80000, 160000, 320000] + + renesas,average-samples: + description: | + Number of samples to be used to report voltage, current and power values. + default: 1 + $ref: /schemas/types.yaml#/definitions/uint32 + enum: [1, 2, 4, 8, 16, 32, 64, 128] + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + i2c { + #address-cells = <1>; + #size-cells = <0>; + + power-monitor@40 { + compatible = "renesas,isl28022"; + reg = <0x40>; + shunt-resistor-micro-ohms = <8000>; + renesas,shunt-range-microvolt = <40000>; + renesas,average-samples = <128>; + }; + }; diff --git a/MAINTAINERS b/MAINTAINERS index b02e3b991676..23b8e8183ece 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11069,6 +11069,7 @@ ISL28022 HARDWARE MONITORING DRIVER M: Carsten Spieß <mail@carsten-spiess.de> L: linux-hwmon@vger.kernel.org S: Maintained +F: Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml F: Documentation/hwmon/isl28022.rst F: drivers/hwmon/isl28022.c -- 2.34.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 2023-08-01 16:35 ` [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 Carsten Spieß @ 2023-08-01 20:52 ` Conor Dooley 2023-08-02 7:30 ` Carsten Spieß 0 siblings, 1 reply; 6+ messages in thread From: Conor Dooley @ 2023-08-01 20:52 UTC (permalink / raw) To: Carsten Spieß Cc: Jean Delvare, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm, linux-hwmon, devicetree, linux-kernel, linux-renesas-soc [-- Attachment #1: Type: text/plain, Size: 3441 bytes --] On Tue, Aug 01, 2023 at 06:35:46PM +0200, Carsten Spieß wrote: > Add dt-bindings for Renesas ISL28022 power monitor. > > Signed-off-by: Carsten Spieß <mail@carsten-spiess.de> > --- > v3: > - changelog added > v2/v3: > - schema errors fixed > - properties reworked > - shunt-resistor minimum and default value added > --- > .../bindings/hwmon/renesas,isl28022.yaml | 65 +++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 66 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml > > diff --git a/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml b/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml > new file mode 100644 > index 000000000000..1e0971287941 > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml > @@ -0,0 +1,65 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/hwmon/renesas,isl28022.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas ISL28022 power monitor > + > +maintainers: > + - Carsten Spieß <mail@carsten-spiess.de> > + > +description: | > + The ISL28022 is a power monitor with I2C interface. The device monitors > + voltage, current via shunt resistor and calculated power. > + > + Datasheets: > + https://www.renesas.com/us/en/www/doc/datasheet/isl28022.pdf > + > +properties: > + compatible: > + enum: > + - renesas,isl28022 You've only got one compatible, why the enum? Will there be more similar devices that have an incompatible programming model? > + > + reg: > + maxItems: 1 > + > + shunt-resistor-micro-ohms: > + description: | > + Shunt resistor value in micro-Ohm > + minimum: 800 > + default: 10000 > + > + renesas,shunt-range-microvolt: > + description: | You don't need these |s if you have no formatting to preserve in the text. Otherwise, this does look good to me. Cheers, Conor. > + Maximal shunt voltage range of +/- 40 mV, 80 mV, 160 mV or 320 mV > + default: 320000 > + enum: [40000, 80000, 160000, 320000] > + > + renesas,average-samples: > + description: | > + Number of samples to be used to report voltage, current and power values. > + default: 1 > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [1, 2, 4, 8, 16, 32, 64, 128] > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + power-monitor@40 { > + compatible = "renesas,isl28022"; > + reg = <0x40>; > + shunt-resistor-micro-ohms = <8000>; > + renesas,shunt-range-microvolt = <40000>; > + renesas,average-samples = <128>; > + }; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index b02e3b991676..23b8e8183ece 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -11069,6 +11069,7 @@ ISL28022 HARDWARE MONITORING DRIVER > M: Carsten Spieß <mail@carsten-spiess.de> > L: linux-hwmon@vger.kernel.org > S: Maintained > +F: Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml > F: Documentation/hwmon/isl28022.rst > F: drivers/hwmon/isl28022.c > > -- > 2.34.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 2023-08-01 20:52 ` Conor Dooley @ 2023-08-02 7:30 ` Carsten Spieß 2023-08-02 7:43 ` Conor Dooley 2023-08-02 15:05 ` Guenter Roeck 0 siblings, 2 replies; 6+ messages in thread From: Carsten Spieß @ 2023-08-02 7:30 UTC (permalink / raw) To: Conor Dooley Cc: Jean Delvare, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm, linux-hwmon, devicetree, linux-kernel, linux-renesas-soc [-- Attachment #1: Type: text/plain, Size: 739 bytes --] On 8/1/23 22:52, Conor Dooley wrote: > On Tue, Aug 01, 2023 at 06:35:46PM +0200, Carsten Spieß wrote: > > Add dt-bindings for Renesas ISL28022 power monitor. > > +properties: > > + compatible: > > + enum: > > + - renesas,isl28022 > > You've only got one compatible, why the enum? Will there be more similar > devices that have an incompatible programming model? Yes, there are isl28023 and isl28025 with different register addresses, might be supported in future releases. > > + renesas,shunt-range-microvolt: > > + description: | > > You don't need these |s if you have no formatting to preserve in the > text. Will fix in v4. > Otherwise, this does look good to me. Thanks, regards Carsten [-- Attachment #2: Digitale Signatur von OpenPGP --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 2023-08-02 7:30 ` Carsten Spieß @ 2023-08-02 7:43 ` Conor Dooley 2023-08-02 15:07 ` Guenter Roeck 2023-08-02 15:05 ` Guenter Roeck 1 sibling, 1 reply; 6+ messages in thread From: Conor Dooley @ 2023-08-02 7:43 UTC (permalink / raw) To: Carsten Spieß Cc: Conor Dooley, Jean Delvare, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm, linux-hwmon, devicetree, linux-kernel, linux-renesas-soc [-- Attachment #1: Type: text/plain, Size: 978 bytes --] On Wed, Aug 02, 2023 at 09:30:23AM +0200, Carsten Spieß wrote: > > On 8/1/23 22:52, Conor Dooley wrote: > > On Tue, Aug 01, 2023 at 06:35:46PM +0200, Carsten Spieß wrote: > > > Add dt-bindings for Renesas ISL28022 power monitor. > > > +properties: > > > + compatible: > > > + enum: > > > + - renesas,isl28022 > > > > You've only got one compatible, why the enum? Will there be more similar > > devices that have an incompatible programming model? > Yes, there are isl28023 and isl28025 with different register addresses, > might be supported in future releases. Right. Unless that's a very strong "might", const: will do the trick here just fine. Otherwise, Reviewed-by: Conor Dooley <conor.dooley@microchip.com> > > > + renesas,shunt-range-microvolt: > > > + description: | > > > > You don't need these |s if you have no formatting to preserve in the > > text. > Will fix in v4. There's no need to send a v4 for that alone. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 2023-08-02 7:43 ` Conor Dooley @ 2023-08-02 15:07 ` Guenter Roeck 0 siblings, 0 replies; 6+ messages in thread From: Guenter Roeck @ 2023-08-02 15:07 UTC (permalink / raw) To: Conor Dooley Cc: Carsten Spieß, Conor Dooley, Jean Delvare, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm, linux-hwmon, devicetree, linux-kernel, linux-renesas-soc On Wed, Aug 02, 2023 at 08:43:59AM +0100, Conor Dooley wrote: > On Wed, Aug 02, 2023 at 09:30:23AM +0200, Carsten Spieß wrote: > > > > On 8/1/23 22:52, Conor Dooley wrote: > > > On Tue, Aug 01, 2023 at 06:35:46PM +0200, Carsten Spieß wrote: > > > > Add dt-bindings for Renesas ISL28022 power monitor. > > > > +properties: > > > > + compatible: > > > > + enum: > > > > + - renesas,isl28022 > > > > > > You've only got one compatible, why the enum? Will there be more similar > > > devices that have an incompatible programming model? > > Yes, there are isl28023 and isl28025 with different register addresses, > > might be supported in future releases. > > Right. Unless that's a very strong "might", const: will do the trick > here just fine. It is a very strong "will never be" for isl28023 and isl28025. Guenter > > Otherwise, > Reviewed-by: Conor Dooley <conor.dooley@microchip.com> > > > > > + renesas,shunt-range-microvolt: > > > > + description: | > > > > > > You don't need these |s if you have no formatting to preserve in the > > > text. > > Will fix in v4. > > There's no need to send a v4 for that alone. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 2023-08-02 7:30 ` Carsten Spieß 2023-08-02 7:43 ` Conor Dooley @ 2023-08-02 15:05 ` Guenter Roeck 1 sibling, 0 replies; 6+ messages in thread From: Guenter Roeck @ 2023-08-02 15:05 UTC (permalink / raw) To: Carsten Spieß Cc: Conor Dooley, Jean Delvare, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven, Magnus Damm, linux-hwmon, devicetree, linux-kernel, linux-renesas-soc On Wed, Aug 02, 2023 at 09:30:23AM +0200, Carsten Spieß wrote: > > On 8/1/23 22:52, Conor Dooley wrote: > > On Tue, Aug 01, 2023 at 06:35:46PM +0200, Carsten Spieß wrote: > > > Add dt-bindings for Renesas ISL28022 power monitor. > > > +properties: > > > + compatible: > > > + enum: > > > + - renesas,isl28022 > > > > You've only got one compatible, why the enum? Will there be more similar > > devices that have an incompatible programming model? > Yes, there are isl28023 and isl28025 with different register addresses, > might be supported in future releases. This is misleading. ISL28023 and ISL28025 are PMBus compatible chips and would be added as PMBus driver(s) (if needed). Support for those chips will never be part of the isl28022 driver, and any devicetree properties of those chips would not be described in this file. Guenter > > > > + renesas,shunt-range-microvolt: > > > + description: | > > > > You don't need these |s if you have no formatting to preserve in the > > text. > Will fix in v4. > > > Otherwise, this does look good to me. > Thanks, regards > Carsten ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-08-02 15:08 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20230801163546.3170-1-mail@carsten-spiess.de>
2023-08-01 16:35 ` [PATCH v3 2/2] dt-bindings: hwmon: add renesas,isl28022 Carsten Spieß
2023-08-01 20:52 ` Conor Dooley
2023-08-02 7:30 ` Carsten Spieß
2023-08-02 7:43 ` Conor Dooley
2023-08-02 15:07 ` Guenter Roeck
2023-08-02 15:05 ` Guenter Roeck
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).