Message ID | c190f764-b3af-8982-00e3-87e28eabecca@gmail.com |
---|---|
State | Changes Requested, archived |
Headers | show |
On Tue, May 30, 2017 at 10:26:33PM +0200, Heiner Kallweit wrote: > Add dt binding documentation for Amlogic meson GPIO interrupt controller. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> > --- > v5: > - added Reviewed-by > --- > .../amlogic,meson-gpio-intc.txt | 26 ++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt > new file mode 100644 > index 00000000..2f20d212 > --- /dev/null > +++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt > @@ -0,0 +1,26 @@ > +Amlogic meson GPIO interrupt controller > + > +Meson SoCs contains an interrupt controller which is able watch the SoC pads > +and generate an interrupt on edges or level. The controller is essentially a > +256 pads to 8 GIC interrupt multiplexer, with a filter block to select edge > +or level and polarity. The actual number of interrupt exposed depends on the > +SoC. > + > +Required properties: > + > +- compatible : should be "amlogic,meson-gpio-intc". > +- reg : Specifies base physical address and size of the registers. > +- interrupt-controller : Identifies the node as an interrupt controller. > +- #interrupt-cells : should be 2. > +- parent-interrupts : list of GIC interrupts which can be used with the > + GPIO IRQ multiplexer Just use "interrupts" here. > + > +Example: > + > +gpio_intc: interrupt-controller@9880 { > + compatible = "amlogic,meson-gpio-intc"; > + reg = <0x0 0x09880 0x0 0x10>; > + interrupt-controller; > + #interrupt-cells = <2>; > + parent-interrupts = <64 65 66 67 68 69 70 71>; > +}; > -- > 2.13.0 > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 06/07/2017 11:13 PM, Rob Herring wrote: > On Tue, May 30, 2017 at 10:26:33PM +0200, Heiner Kallweit wrote: >> Add dt binding documentation for Amlogic meson GPIO interrupt controller. >> >> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> >> Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> >> --- >> v5: >> - added Reviewed-by >> --- >> .../amlogic,meson-gpio-intc.txt | 26 ++++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >> >> diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >> new file mode 100644 >> index 00000000..2f20d212 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >> @@ -0,0 +1,26 @@ >> +Amlogic meson GPIO interrupt controller >> + >> +Meson SoCs contains an interrupt controller which is able watch the SoC pads >> +and generate an interrupt on edges or level. The controller is essentially a >> +256 pads to 8 GIC interrupt multiplexer, with a filter block to select edge >> +or level and polarity. The actual number of interrupt exposed depends on the >> +SoC. >> + >> +Required properties: >> + >> +- compatible : should be "amlogic,meson-gpio-intc". >> +- reg : Specifies base physical address and size of the registers. >> +- interrupt-controller : Identifies the node as an interrupt controller. >> +- #interrupt-cells : should be 2. >> +- parent-interrupts : list of GIC interrupts which can be used with the >> + GPIO IRQ multiplexer > > Just use "interrupts" here. Hi Rob, If the "interrupts" property is used here like this, it breaks the interrupts binding ! Neil > >> + >> +Example: >> + >> +gpio_intc: interrupt-controller@9880 { >> + compatible = "amlogic,meson-gpio-intc"; >> + reg = <0x0 0x09880 0x0 0x10>; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + parent-interrupts = <64 65 66 67 68 69 70 71>; >> +}; >> -- >> 2.13.0 >> >> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Jun 9, 2017 at 2:50 AM, Neil Armstrong <narmstrong@baylibre.com> wrote: > On 06/07/2017 11:13 PM, Rob Herring wrote: >> On Tue, May 30, 2017 at 10:26:33PM +0200, Heiner Kallweit wrote: >>> Add dt binding documentation for Amlogic meson GPIO interrupt controller. >>> >>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> >>> Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> >>> --- >>> v5: >>> - added Reviewed-by >>> --- >>> .../amlogic,meson-gpio-intc.txt | 26 ++++++++++++++++++++++ >>> 1 file changed, 26 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >>> >>> diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >>> new file mode 100644 >>> index 00000000..2f20d212 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt >>> @@ -0,0 +1,26 @@ >>> +Amlogic meson GPIO interrupt controller >>> + >>> +Meson SoCs contains an interrupt controller which is able watch the SoC pads >>> +and generate an interrupt on edges or level. The controller is essentially a >>> +256 pads to 8 GIC interrupt multiplexer, with a filter block to select edge >>> +or level and polarity. The actual number of interrupt exposed depends on the >>> +SoC. >>> + >>> +Required properties: >>> + >>> +- compatible : should be "amlogic,meson-gpio-intc". >>> +- reg : Specifies base physical address and size of the registers. >>> +- interrupt-controller : Identifies the node as an interrupt controller. >>> +- #interrupt-cells : should be 2. >>> +- parent-interrupts : list of GIC interrupts which can be used with the >>> + GPIO IRQ multiplexer >> >> Just use "interrupts" here. > > Hi Rob, > > If the "interrupts" property is used here like this, it breaks the interrupts binding ! Really!? Looking closer at the series, in fact the dts files are already correct: + gpio_intc: interrupt-controller@9880 { + compatible = "amlogic,meson-gpio-intc"; + reg = <0x9880 0x10>; + interrupt-controller; + #interrupt-cells = <2>; + interrupts = <64 65 66 67 68 79 70 71>; + }; + Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt new file mode 100644 index 00000000..2f20d212 --- /dev/null +++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt @@ -0,0 +1,26 @@ +Amlogic meson GPIO interrupt controller + +Meson SoCs contains an interrupt controller which is able watch the SoC pads +and generate an interrupt on edges or level. The controller is essentially a +256 pads to 8 GIC interrupt multiplexer, with a filter block to select edge +or level and polarity. The actual number of interrupt exposed depends on the +SoC. + +Required properties: + +- compatible : should be "amlogic,meson-gpio-intc". +- reg : Specifies base physical address and size of the registers. +- interrupt-controller : Identifies the node as an interrupt controller. +- #interrupt-cells : should be 2. +- parent-interrupts : list of GIC interrupts which can be used with the + GPIO IRQ multiplexer + +Example: + +gpio_intc: interrupt-controller@9880 { + compatible = "amlogic,meson-gpio-intc"; + reg = <0x0 0x09880 0x0 0x10>; + interrupt-controller; + #interrupt-cells = <2>; + parent-interrupts = <64 65 66 67 68 69 70 71>; +};