| Message ID | 20211011155615.257529-6-luca@lucaceresoli.net |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | Add MAX77714 PMIC minimal driver (RTC and watchdog only) | expand |
On 11/10/2021 17:56, Luca Ceresoli wrote: > Add bindings for the MAX77714 PMIC with GPIO, RTC and watchdog. > > Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> > --- > .../bindings/mfd/maxim,max77714.yaml | 58 +++++++++++++++++++ > MAINTAINERS | 5 ++ > 2 files changed, 63 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77714.yaml > > diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml > new file mode 100644 > index 000000000000..2b0ce3b9bc92 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mfd/maxim,max77714.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: MAX77714 PMIC with GPIO, RTC and watchdog from Maxim Integrated. > + > +maintainers: > + - Luca Ceresoli <luca@lucaceresoli.net> > + > +description: | > + MAX77714 is a Power Management IC with 4 buck regulators, 9 > + low-dropout regulators, 8 GPIOs, RTC and watchdog. > + > +properties: > + compatible: > + const: maxim,max77714 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + interrupt-controller: true > + > + "#interrupt-cells": > + const: 2 > + description: > + The first cell is the IRQ number, the second cell is the trigger type. > + > +required: > + - compatible > + - reg > + - interrupts > + - interrupt-controller > + - "#interrupt-cells" > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + pmic@1c { > + compatible = "maxim,max77714"; > + reg = <0x1c>; > + interrupt-parent = <&gpio2>; > + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; > + > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > + }; Looks good to me, but what about regulators and other properties? Are you planning to add them later? Best regards, Krzysztof
Hi Krzysztof, thanks for reviewing. On 12/10/21 10:02, Krzysztof Kozlowski wrote: > On 11/10/2021 17:56, Luca Ceresoli wrote: >> Add bindings for the MAX77714 PMIC with GPIO, RTC and watchdog. >> >> Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> >> --- >> .../bindings/mfd/maxim,max77714.yaml | 58 +++++++++++++++++++ >> MAINTAINERS | 5 ++ >> 2 files changed, 63 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >> >> diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >> new file mode 100644 >> index 000000000000..2b0ce3b9bc92 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >> @@ -0,0 +1,58 @@ >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/mfd/maxim,max77714.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: MAX77714 PMIC with GPIO, RTC and watchdog from Maxim Integrated. >> + >> +maintainers: >> + - Luca Ceresoli <luca@lucaceresoli.net> >> + >> +description: | >> + MAX77714 is a Power Management IC with 4 buck regulators, 9 >> + low-dropout regulators, 8 GPIOs, RTC and watchdog. >> + >> +properties: >> + compatible: >> + const: maxim,max77714 >> + >> + reg: >> + maxItems: 1 >> + >> + interrupts: >> + maxItems: 1 >> + >> + interrupt-controller: true >> + >> + "#interrupt-cells": >> + const: 2 >> + description: >> + The first cell is the IRQ number, the second cell is the trigger type. >> + >> +required: >> + - compatible >> + - reg >> + - interrupts >> + - interrupt-controller >> + - "#interrupt-cells" >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + #include <dt-bindings/interrupt-controller/irq.h> >> + i2c { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + pmic@1c { >> + compatible = "maxim,max77714"; >> + reg = <0x1c>; >> + interrupt-parent = <&gpio2>; >> + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; >> + >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + }; >> + }; > > Looks good to me, but what about regulators and other properties? Are > you planning to add them later? No plan to add them, sorry. I know, complete bindings are better than incomplete bindings. But in the foreseeable future I don't need to do anything on the regulators (even though it might happen at some point). And since their setting is possibly non trivial, I'm not going to study them to write a complete bindings document and then make no use of it. Is it a problem for you?
On 13/10/2021 23:28, Luca Ceresoli wrote: > Hi Krzysztof, > > thanks for reviewing. > > On 12/10/21 10:02, Krzysztof Kozlowski wrote: >> On 11/10/2021 17:56, Luca Ceresoli wrote: >>> Add bindings for the MAX77714 PMIC with GPIO, RTC and watchdog. >>> >>> Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> >>> --- >>> .../bindings/mfd/maxim,max77714.yaml | 58 +++++++++++++++++++ >>> MAINTAINERS | 5 ++ >>> 2 files changed, 63 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >>> new file mode 100644 >>> index 000000000000..2b0ce3b9bc92 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml >>> @@ -0,0 +1,58 @@ >>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/mfd/maxim,max77714.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: MAX77714 PMIC with GPIO, RTC and watchdog from Maxim Integrated. >>> + >>> +maintainers: >>> + - Luca Ceresoli <luca@lucaceresoli.net> >>> + >>> +description: | >>> + MAX77714 is a Power Management IC with 4 buck regulators, 9 >>> + low-dropout regulators, 8 GPIOs, RTC and watchdog. >>> + >>> +properties: >>> + compatible: >>> + const: maxim,max77714 >>> + >>> + reg: >>> + maxItems: 1 >>> + >>> + interrupts: >>> + maxItems: 1 >>> + >>> + interrupt-controller: true >>> + >>> + "#interrupt-cells": >>> + const: 2 >>> + description: >>> + The first cell is the IRQ number, the second cell is the trigger type. >>> + >>> +required: >>> + - compatible >>> + - reg >>> + - interrupts >>> + - interrupt-controller >>> + - "#interrupt-cells" >>> + >>> +additionalProperties: false >>> + >>> +examples: >>> + - | >>> + #include <dt-bindings/interrupt-controller/irq.h> >>> + i2c { >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + >>> + pmic@1c { >>> + compatible = "maxim,max77714"; >>> + reg = <0x1c>; >>> + interrupt-parent = <&gpio2>; >>> + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; >>> + >>> + interrupt-controller; >>> + #interrupt-cells = <2>; >>> + }; >>> + }; >> >> Looks good to me, but what about regulators and other properties? Are >> you planning to add them later? > > No plan to add them, sorry. > > I know, complete bindings are better than incomplete bindings. But in > the foreseeable future I don't need to do anything on the regulators > (even though it might happen at some point). And since their setting is > possibly non trivial, I'm not going to study them to write a complete > bindings document and then make no use of it. > > Is it a problem for you? It's OK. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml new file mode 100644 index 000000000000..2b0ce3b9bc92 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/maxim,max77714.yaml @@ -0,0 +1,58 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/mfd/maxim,max77714.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: MAX77714 PMIC with GPIO, RTC and watchdog from Maxim Integrated. + +maintainers: + - Luca Ceresoli <luca@lucaceresoli.net> + +description: | + MAX77714 is a Power Management IC with 4 buck regulators, 9 + low-dropout regulators, 8 GPIOs, RTC and watchdog. + +properties: + compatible: + const: maxim,max77714 + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + interrupt-controller: true + + "#interrupt-cells": + const: 2 + description: + The first cell is the IRQ number, the second cell is the trigger type. + +required: + - compatible + - reg + - interrupts + - interrupt-controller + - "#interrupt-cells" + +additionalProperties: false + +examples: + - | + #include <dt-bindings/interrupt-controller/irq.h> + i2c { + #address-cells = <1>; + #size-cells = <0>; + + pmic@1c { + compatible = "maxim,max77714"; + reg = <0x1c>; + interrupt-parent = <&gpio2>; + interrupts = <3 IRQ_TYPE_LEVEL_LOW>; + + interrupt-controller; + #interrupt-cells = <2>; + }; + }; diff --git a/MAINTAINERS b/MAINTAINERS index a4a0c2baaf27..4d0134752537 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11385,6 +11385,11 @@ F: drivers/power/supply/max77650-charger.c F: drivers/regulator/max77650-regulator.c F: include/linux/mfd/max77650.h +MAXIM MAX77714 PMIC MFD DRIVER +M: Luca Ceresoli <luca@lucaceresoli.net> +S: Maintained +F: Documentation/devicetree/bindings/mfd/maxim,max77714.yaml + MAXIM MAX77802 PMIC REGULATOR DEVICE DRIVER M: Javier Martinez Canillas <javier@dowhile0.org> L: linux-kernel@vger.kernel.org
Add bindings for the MAX77714 PMIC with GPIO, RTC and watchdog. Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> --- .../bindings/mfd/maxim,max77714.yaml | 58 +++++++++++++++++++ MAINTAINERS | 5 ++ 2 files changed, 63 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77714.yaml