| Message ID | 20230601054549.10843-15-nikita.shubin@maquefel.me |
|---|---|
| State | Accepted |
| Commit | a1b78312ec440e8b4c633bb4985dd825abd29488 |
| Headers | show |
| Series | ep93xx device tree conversion | expand |
On 01/06/2023 07:45, Nikita Shubin wrote: > Add YAML bindings for ep93xx SoC gpio controller. > > Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me> > --- > .../devicetree/bindings/gpio/gpio-ep9301.yaml | 154 ++++++++++++++++++ > 1 file changed, 154 insertions(+) > create mode 100644 Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml b/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml > new file mode 100644 > index 000000000000..daadfb4926c3 > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml Filename... > @@ -0,0 +1,154 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/gpio/gpio-ep9301.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: EP93xx GPIO controller > + > +maintainers: > + - Linus Walleij <linus.walleij@linaro.org> > + - Bartosz Golaszewski <brgl@bgdev.pl> Did you choose correct maintainers? Bartosz, Linus, do you take care about EP93xx platform? > + - Nikita Shubin <nikita.shubin@maquefel.me> > + > +properties: > + compatible: > + oneOf: > + - const: cirrus,ep9301-gpio > + - items: > + - enum: > + - cirrus,ep9302-gpio > + - cirrus,ep9307-gpio > + - cirrus,ep9312-gpio > + - cirrus,ep9315-gpio > + - const: cirrus,ep9301-gpio > + > + reg: > + minItems: 2 > + items: > + - description: data register > + - description: direction register > + - description: interrupt registers base > + > + reg-names: > + minItems: 2 > + items: > + - const: data > + - const: dir > + - const: intr > + > + gpio-controller: true > + > + gpio-ranges: true > + > + "#gpio-cells": > + const: 2 > + > + interrupt-controller: true > + > + "#interrupt-cells": > + const: 2 > + > + interrupts: > + oneOf: > + - maxItems: 1 > + - description: port F has dedicated irq line for each gpio line > + maxItems: 8 > + > +required: > + - compatible > + - reg > + - gpio-controller > + - "#gpio-cells" > + > +additionalProperties: false > + > +examples: > + - | > + gpio@80840000 { > + compatible = "cirrus,ep9301-gpio"; > + reg = <0x80840000 0x04>, > + <0x80840010 0x04>, > + <0x80840090 0x1c>; > + reg-names = "data", "dir", "intr"; > + gpio-controller; > + #gpio-cells = <2>; > + interrupt-controller; > + interrupt-parent = <&vic1>; > + interrupts = <27>; > + }; > + > + gpio@80840004 { > + compatible = "cirrus,ep9301-gpio"; > + reg = <0x80840004 0x04>, > + <0x80840014 0x04>, > + <0x808400ac 0x1c>; > + reg-names = "data", "dir", "intr"; > + gpio-controller; > + #gpio-cells = <2>; > + interrupt-controller; > + interrupt-parent = <&vic1>; > + interrupts = <27>; > + }; Same example as before, drop. > + > + gpio@80840008 { > + compatible = "cirrus,ep9301-gpio"; > + reg = <0x80840008 0x04>, > + <0x80840018 0x04>; > + reg-names = "data", "dir"; > + gpio-controller; > + #gpio-cells = <2>; > + }; > + > + gpio@8084000c { > + compatible = "cirrus,ep9301-gpio"; > + reg = <0x8084000c 0x04>, > + <0x8084001c 0x04>; > + reg-names = "data", "dir"; > + gpio-controller; > + #gpio-cells = <2>; > + }; Same as before... Just keep one example for interrupt controller and one for non-interrupt-controller. Best regards, Krzysztof
On Thu, Jun 1, 2023 at 10:20 AM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > +title: EP93xx GPIO controller > > + > > +maintainers: > > + - Linus Walleij <linus.walleij@linaro.org> > > + - Bartosz Golaszewski <brgl@bgdev.pl> > > Did you choose correct maintainers? Bartosz, Linus, do you take care > about EP93xx platform? I'm fine with it (I have a platform). Yours, Linus Walleij
On Fri, Jun 2, 2023 at 9:41 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > On Thu, Jun 1, 2023 at 10:20 AM Krzysztof Kozlowski > <krzysztof.kozlowski@linaro.org> wrote: > > > > +title: EP93xx GPIO controller > > > + > > > +maintainers: > > > + - Linus Walleij <linus.walleij@linaro.org> > > > + - Bartosz Golaszewski <brgl@bgdev.pl> > > > > Did you choose correct maintainers? Bartosz, Linus, do you take care > > about EP93xx platform? > > I'm fine with it (I have a platform). I don't but I'm actually not sure how DT bindings maintainership works - do GPIO bindings all fall under the GPIO jurisdiction automatically? Bart
On Tue, Jun 13, 2023 at 4:55 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > On Fri, Jun 2, 2023 at 9:41 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > On Thu, Jun 1, 2023 at 10:20 AM Krzysztof Kozlowski > > <krzysztof.kozlowski@linaro.org> wrote: > > > Did you choose correct maintainers? Bartosz, Linus, do you take care > > > about EP93xx platform? > > > > I'm fine with it (I have a platform). > > I don't but I'm actually not sure how DT bindings maintainership works > - do GPIO bindings all fall under the GPIO jurisdiction automatically? Not really, more on the people selected by the person writing the bindings, confirmed by them being merged. Traditionally, Linux as the biggest software project with the most active subsystem maintainers do the reviewing and take the responsibility for them. Technically, e.g. BSD people could appear on the devicetree mailinglist and review patches and suggest maintainership from their side, but I haven't seen them much around. Neither Apple Computer or anyone else who ought to be there but isn't. Yours, Linus Walleij
On Thu, Jun 1, 2023 at 7:46 AM Nikita Shubin <nikita.shubin@maquefel.me> wrote: > > Add YAML bindings for ep93xx SoC gpio controller. > > Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me> > --- Applied, thanks! Bart
diff --git a/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml b/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml new file mode 100644 index 000000000000..daadfb4926c3 --- /dev/null +++ b/Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml @@ -0,0 +1,154 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/gpio/gpio-ep9301.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: EP93xx GPIO controller + +maintainers: + - Linus Walleij <linus.walleij@linaro.org> + - Bartosz Golaszewski <brgl@bgdev.pl> + - Nikita Shubin <nikita.shubin@maquefel.me> + +properties: + compatible: + oneOf: + - const: cirrus,ep9301-gpio + - items: + - enum: + - cirrus,ep9302-gpio + - cirrus,ep9307-gpio + - cirrus,ep9312-gpio + - cirrus,ep9315-gpio + - const: cirrus,ep9301-gpio + + reg: + minItems: 2 + items: + - description: data register + - description: direction register + - description: interrupt registers base + + reg-names: + minItems: 2 + items: + - const: data + - const: dir + - const: intr + + gpio-controller: true + + gpio-ranges: true + + "#gpio-cells": + const: 2 + + interrupt-controller: true + + "#interrupt-cells": + const: 2 + + interrupts: + oneOf: + - maxItems: 1 + - description: port F has dedicated irq line for each gpio line + maxItems: 8 + +required: + - compatible + - reg + - gpio-controller + - "#gpio-cells" + +additionalProperties: false + +examples: + - | + gpio@80840000 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840000 0x04>, + <0x80840010 0x04>, + <0x80840090 0x1c>; + reg-names = "data", "dir", "intr"; + gpio-controller; + #gpio-cells = <2>; + interrupt-controller; + interrupt-parent = <&vic1>; + interrupts = <27>; + }; + + gpio@80840004 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840004 0x04>, + <0x80840014 0x04>, + <0x808400ac 0x1c>; + reg-names = "data", "dir", "intr"; + gpio-controller; + #gpio-cells = <2>; + interrupt-controller; + interrupt-parent = <&vic1>; + interrupts = <27>; + }; + + gpio@80840008 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840008 0x04>, + <0x80840018 0x04>; + reg-names = "data", "dir"; + gpio-controller; + #gpio-cells = <2>; + }; + + gpio@8084000c { + compatible = "cirrus,ep9301-gpio"; + reg = <0x8084000c 0x04>, + <0x8084001c 0x04>; + reg-names = "data", "dir"; + gpio-controller; + #gpio-cells = <2>; + }; + + gpio@80840020 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840020 0x04>, + <0x80840024 0x04>; + reg-names = "data", "dir"; + gpio-controller; + #gpio-cells = <2>; + }; + + gpio@80840030 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840030 0x04>, + <0x80840034 0x04>, + <0x8084004c 0x1c>; + reg-names = "data", "dir", "intr"; + gpio-controller; + #gpio-cells = <2>; + interrupt-controller; + interrupts-extended = <&vic0 19>, <&vic0 20>, + <&vic0 21>, <&vic0 22>, + <&vic1 15>, <&vic1 16>, + <&vic1 17>, <&vic1 18>; + }; + + gpio@80840038 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840038 0x04>, + <0x8084003c 0x04>; + reg-names = "data", "dir"; + gpio-controller; + #gpio-cells = <2>; + }; + + gpio@80840040 { + compatible = "cirrus,ep9301-gpio"; + reg = <0x80840040 0x04>, + <0x80840044 0x04>; + reg-names = "data", "dir"; + gpio-controller; + #gpio-cells = <2>; + }; + +...
Add YAML bindings for ep93xx SoC gpio controller. Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me> --- .../devicetree/bindings/gpio/gpio-ep9301.yaml | 154 ++++++++++++++++++ 1 file changed, 154 insertions(+) create mode 100644 Documentation/devicetree/bindings/gpio/gpio-ep9301.yaml