Re: [PATCH v2 14/14] ARM: dts: AM43x: initial support
On 05/29/2013 10:57 AM, Florian Vaussard wrote: > Hello, > > On 05/29/2013 10:53 AM, Benoit Cousson wrote: >> + Florian >> >> Hi Afzal, >> >> On 05/27/2013 04:37 PM, Afzal Mohammed wrote: >>> DT source (minimal) for AM4372 SoC to represent AM43x SoC's. Those >>> represented here are the minimal DT nodes necessary to get kernel >>> booting. >>> >>> In DT nodes, "ti,hwmod" property has not been added, this would be >>> added along with PRCM support for AM43x. >>> >>> Signed-off-by: Ankur Kishore >>> Signed-off-by: Afzal Mohammed >>> --- >>> >>> v2: Add gptimer 1ms, timer2, synctimer and remove twd local timer >>> >>> arch/arm/boot/dts/am4372.dtsi | 66 >>> +++ >>> 1 file changed, 66 insertions(+) >>> create mode 100644 arch/arm/boot/dts/am4372.dtsi >>> >>> diff --git a/arch/arm/boot/dts/am4372.dtsi >>> b/arch/arm/boot/dts/am4372.dtsi >>> new file mode 100644 >>> index 000..1d58298 >>> --- /dev/null >>> +++ b/arch/arm/boot/dts/am4372.dtsi >>> @@ -0,0 +1,66 @@ >>> +/* >>> + * Device Tree Source for AM4372 SoC >>> + * >>> + * Copyright (C) 2013 Texas Instruments Incorporated - >>> http://www.ti.com/ >>> + * >>> + * This file is licensed under the terms of the GNU General Public >>> License >>> + * version 2. This program is licensed "as is" without any warranty >>> of any >>> + * kind, whether express or implied. >>> + */ >>> + >>> +/include/ "skeleton.dtsi" >> >> You can now use the C preprocessor statement instead of this one. >> Florian already started doing the change [1]. >> >> Beside that detail, that patch looks good to me. >> I'll pull it separately of the series. >> > > If you pull the patch in your branch, I can take care of the changes > when I rebase > my series. This will allow me to clean the 'interrupts' statements below > as well. Ooops, thanks, I missed that one. Thanks, Benoit > > Regards, > > Florian > >> Regards, >> Benoit >> >> [1] http://thread.gmane.org/gmane.linux.ports.arm.omap/98320 >> >>> + >>> +/ { >>> +compatible = "ti,am4372", "ti,am43"; >>> +interrupt-parent = <&gic>; >>> + >>> + >>> +aliases { >>> +serial0 = &uart1; >>> +}; >>> + >>> +cpus { >>> +cpu@0 { >>> +compatible = "arm,cortex-a9"; >>> +}; >>> +}; >>> + >>> +gic: interrupt-controller@48241000 { >>> +compatible = "arm,cortex-a9-gic"; >>> +interrupt-controller; >>> +#interrupt-cells = <3>; >>> +reg = <0x48241000 0x1000>, >>> + <0x48240100 0x0100>; >>> +}; >>> + >>> +ocp { >>> +compatible = "simple-bus"; >>> +#address-cells = <1>; >>> +#size-cells = <1>; >>> +ranges; >>> + >>> +uart1: serial@44e09000 { >>> +compatible = "ti,am4372-uart","ti,omap2-uart"; >>> +reg = <0x44e09000 0x2000>; >>> +interrupts = <0 72 0x4>; >>> +}; >>> + >>> +timer1: timer@44e31000 { >>> +compatible = "ti,am4372-timer-1ms","ti,am335x-timer-1ms"; >>> +reg = <0x44e31000 0x400>; >>> +interrupts = <0 67 0x4>; >>> +ti,timer-alwon; >>> +}; >>> + >>> +timer2: timer@4804 { >>> +compatible = "ti,am4372-timer","ti,am335x-timer"; >>> +reg = <0x4804 0x400>; >>> +interrupts = <0 68 0x4>; >>> +}; >>> + >>> +counter32k: counter@44e86000 { >>> +compatible = "ti,am4372-counter32k","ti,omap-counter32k"; >>> +reg = <0x44e86000 0x40>; >>> +}; >>> +}; >>> +}; >>> >> ___ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss
Re: [PATCH v2 14/14] ARM: dts: AM43x: initial support
Hello, On 05/29/2013 10:53 AM, Benoit Cousson wrote: + Florian Hi Afzal, On 05/27/2013 04:37 PM, Afzal Mohammed wrote: DT source (minimal) for AM4372 SoC to represent AM43x SoC's. Those represented here are the minimal DT nodes necessary to get kernel booting. In DT nodes, "ti,hwmod" property has not been added, this would be added along with PRCM support for AM43x. Signed-off-by: Ankur Kishore Signed-off-by: Afzal Mohammed --- v2: Add gptimer 1ms, timer2, synctimer and remove twd local timer arch/arm/boot/dts/am4372.dtsi | 66 +++ 1 file changed, 66 insertions(+) create mode 100644 arch/arm/boot/dts/am4372.dtsi diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi new file mode 100644 index 000..1d58298 --- /dev/null +++ b/arch/arm/boot/dts/am4372.dtsi @@ -0,0 +1,66 @@ +/* + * Device Tree Source for AM4372 SoC + * + * Copyright (C) 2013 Texas Instruments Incorporated - http://www.ti.com/ + * + * This file is licensed under the terms of the GNU General Public License + * version 2. This program is licensed "as is" without any warranty of any + * kind, whether express or implied. + */ + +/include/ "skeleton.dtsi" You can now use the C preprocessor statement instead of this one. Florian already started doing the change [1]. Beside that detail, that patch looks good to me. I'll pull it separately of the series. If you pull the patch in your branch, I can take care of the changes when I rebase my series. This will allow me to clean the 'interrupts' statements below as well. Regards, Florian Regards, Benoit [1] http://thread.gmane.org/gmane.linux.ports.arm.omap/98320 + +/ { + compatible = "ti,am4372", "ti,am43"; + interrupt-parent = <&gic>; + + + aliases { + serial0 = &uart1; + }; + + cpus { + cpu@0 { + compatible = "arm,cortex-a9"; + }; + }; + + gic: interrupt-controller@48241000 { + compatible = "arm,cortex-a9-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0x48241000 0x1000>, + <0x48240100 0x0100>; + }; + + ocp { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges; + + uart1: serial@44e09000 { + compatible = "ti,am4372-uart","ti,omap2-uart"; + reg = <0x44e09000 0x2000>; + interrupts = <0 72 0x4>; + }; + + timer1: timer@44e31000 { + compatible = "ti,am4372-timer-1ms","ti,am335x-timer-1ms"; + reg = <0x44e31000 0x400>; + interrupts = <0 67 0x4>; + ti,timer-alwon; + }; + + timer2: timer@4804 { + compatible = "ti,am4372-timer","ti,am335x-timer"; + reg = <0x4804 0x400>; + interrupts = <0 68 0x4>; + }; + + counter32k: counter@44e86000 { + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; + reg = <0x44e86000 0x40>; + }; + }; +}; ___ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss
Re: [PATCH v2 14/14] ARM: dts: AM43x: initial support
+ Florian Hi Afzal, On 05/27/2013 04:37 PM, Afzal Mohammed wrote: > DT source (minimal) for AM4372 SoC to represent AM43x SoC's. Those > represented here are the minimal DT nodes necessary to get kernel > booting. > > In DT nodes, "ti,hwmod" property has not been added, this would be > added along with PRCM support for AM43x. > > Signed-off-by: Ankur Kishore > Signed-off-by: Afzal Mohammed > --- > > v2: Add gptimer 1ms, timer2, synctimer and remove twd local timer > > arch/arm/boot/dts/am4372.dtsi | 66 > +++ > 1 file changed, 66 insertions(+) > create mode 100644 arch/arm/boot/dts/am4372.dtsi > > diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi > new file mode 100644 > index 000..1d58298 > --- /dev/null > +++ b/arch/arm/boot/dts/am4372.dtsi > @@ -0,0 +1,66 @@ > +/* > + * Device Tree Source for AM4372 SoC > + * > + * Copyright (C) 2013 Texas Instruments Incorporated - http://www.ti.com/ > + * > + * This file is licensed under the terms of the GNU General Public License > + * version 2. This program is licensed "as is" without any warranty of any > + * kind, whether express or implied. > + */ > + > +/include/ "skeleton.dtsi" You can now use the C preprocessor statement instead of this one. Florian already started doing the change [1]. Beside that detail, that patch looks good to me. I'll pull it separately of the series. Regards, Benoit [1] http://thread.gmane.org/gmane.linux.ports.arm.omap/98320 > + > +/ { > + compatible = "ti,am4372", "ti,am43"; > + interrupt-parent = <&gic>; > + > + > + aliases { > + serial0 = &uart1; > + }; > + > + cpus { > + cpu@0 { > + compatible = "arm,cortex-a9"; > + }; > + }; > + > + gic: interrupt-controller@48241000 { > + compatible = "arm,cortex-a9-gic"; > + interrupt-controller; > + #interrupt-cells = <3>; > + reg = <0x48241000 0x1000>, > + <0x48240100 0x0100>; > + }; > + > + ocp { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + uart1: serial@44e09000 { > + compatible = "ti,am4372-uart","ti,omap2-uart"; > + reg = <0x44e09000 0x2000>; > + interrupts = <0 72 0x4>; > + }; > + > + timer1: timer@44e31000 { > + compatible = > "ti,am4372-timer-1ms","ti,am335x-timer-1ms"; > + reg = <0x44e31000 0x400>; > + interrupts = <0 67 0x4>; > + ti,timer-alwon; > + }; > + > + timer2: timer@4804 { > + compatible = "ti,am4372-timer","ti,am335x-timer"; > + reg = <0x4804 0x400>; > + interrupts = <0 68 0x4>; > + }; > + > + counter32k: counter@44e86000 { > + compatible = > "ti,am4372-counter32k","ti,omap-counter32k"; > + reg = <0x44e86000 0x40>; > + }; > + }; > +}; > ___ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss
[PATCH v2 14/14] ARM: dts: AM43x: initial support
DT source (minimal) for AM4372 SoC to represent AM43x SoC's. Those represented here are the minimal DT nodes necessary to get kernel booting. In DT nodes, "ti,hwmod" property has not been added, this would be added along with PRCM support for AM43x. Signed-off-by: Ankur Kishore Signed-off-by: Afzal Mohammed --- v2: Add gptimer 1ms, timer2, synctimer and remove twd local timer arch/arm/boot/dts/am4372.dtsi | 66 +++ 1 file changed, 66 insertions(+) create mode 100644 arch/arm/boot/dts/am4372.dtsi diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi new file mode 100644 index 000..1d58298 --- /dev/null +++ b/arch/arm/boot/dts/am4372.dtsi @@ -0,0 +1,66 @@ +/* + * Device Tree Source for AM4372 SoC + * + * Copyright (C) 2013 Texas Instruments Incorporated - http://www.ti.com/ + * + * This file is licensed under the terms of the GNU General Public License + * version 2. This program is licensed "as is" without any warranty of any + * kind, whether express or implied. + */ + +/include/ "skeleton.dtsi" + +/ { + compatible = "ti,am4372", "ti,am43"; + interrupt-parent = <&gic>; + + + aliases { + serial0 = &uart1; + }; + + cpus { + cpu@0 { + compatible = "arm,cortex-a9"; + }; + }; + + gic: interrupt-controller@48241000 { + compatible = "arm,cortex-a9-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0x48241000 0x1000>, + <0x48240100 0x0100>; + }; + + ocp { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges; + + uart1: serial@44e09000 { + compatible = "ti,am4372-uart","ti,omap2-uart"; + reg = <0x44e09000 0x2000>; + interrupts = <0 72 0x4>; + }; + + timer1: timer@44e31000 { + compatible = "ti,am4372-timer-1ms","ti,am335x-timer-1ms"; + reg = <0x44e31000 0x400>; + interrupts = <0 67 0x4>; + ti,timer-alwon; + }; + + timer2: timer@4804 { + compatible = "ti,am4372-timer","ti,am335x-timer"; + reg = <0x4804 0x400>; + interrupts = <0 68 0x4>; + }; + + counter32k: counter@44e86000 { + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; + reg = <0x44e86000 0x40>; + }; + }; +}; -- 1.7.12 ___ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss