Re: [RFC PATCH 1/3] arm64: dts: ti: am6-wakeup: Add gpio node
On 05/06/19 11:46 AM, Lokesh Vutla wrote: On 05/06/19 11:38 AM, Keerthy wrote: Add gpio0 node under wakeup domain. This has 56 gpios and all are capable of generating banked interrupts. Signed-off-by: Keerthy --- arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 15 +++ 1 file changed, 15 insertions(+) diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi index f1ca171abdf8..8c6c99e7c6ed 100644 --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi @@ -74,4 +74,19 @@ ti,sci-dst-id = <56>; ti,sci-rm-range-girq = <0x4>; }; + + wkup_gpio0: wkup_gpio0@4211 { + compatible = "ti,k2g-gpio", "ti,keystone-gpio"; This is not k2g. Can you either create a am6 specific compatible or just use ti,keystone-gpio. It seems practice is now to have separate compatible. I will add am6 specific compatible. Thanks and regards, Lokesh + reg = <0x4211 0x100>; + gpio-controller; + #gpio-cells = <2>; + interrupt-parent = <_wkup_gpio>; + interrupts = <59 128>, <59 129>, <59 130>, <59 131>; + interrupt-controller; + #interrupt-cells = <2>; + ti,ngpio = <56>; + ti,davinci-gpio-unbanked = <0>; + clocks = <_clks 59 0>; + clock-names = "gpio"; + }; };
Re: [RFC PATCH 1/3] arm64: dts: ti: am6-wakeup: Add gpio node
On 05/06/19 11:38 AM, Keerthy wrote: > Add gpio0 node under wakeup domain. This has 56 gpios > and all are capable of generating banked interrupts. > > Signed-off-by: Keerthy > --- > arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > index f1ca171abdf8..8c6c99e7c6ed 100644 > --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > @@ -74,4 +74,19 @@ > ti,sci-dst-id = <56>; > ti,sci-rm-range-girq = <0x4>; > }; > + > + wkup_gpio0: wkup_gpio0@4211 { > + compatible = "ti,k2g-gpio", "ti,keystone-gpio"; This is not k2g. Can you either create a am6 specific compatible or just use ti,keystone-gpio. Thanks and regards, Lokesh > + reg = <0x4211 0x100>; > + gpio-controller; > + #gpio-cells = <2>; > + interrupt-parent = <_wkup_gpio>; > + interrupts = <59 128>, <59 129>, <59 130>, <59 131>; > + interrupt-controller; > + #interrupt-cells = <2>; > + ti,ngpio = <56>; > + ti,davinci-gpio-unbanked = <0>; > + clocks = <_clks 59 0>; > + clock-names = "gpio"; > + }; > }; >