| Message ID | 3790fa57fc4c69da12224fbcec17cd07c59771a0.1369658705.git.afzal@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
+ 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 <a-kishore@ti.com> > Signed-off-by: Afzal Mohammed <afzal@ti.com> > --- > > 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 0000000..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@48040000 { > + compatible = "ti,am4372-timer","ti,am335x-timer"; > + reg = <0x48040000 0x400>; > + interrupts = <0 68 0x4>; > + }; > + > + counter32k: counter@44e86000 { > + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; > + reg = <0x44e86000 0x40>; > + }; > + }; > +}; > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 <a-kishore@ti.com> >> Signed-off-by: Afzal Mohammed <afzal@ti.com> >> --- >> >> 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 0000000..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@48040000 { >> + compatible = "ti,am4372-timer","ti,am335x-timer"; >> + reg = <0x48040000 0x400>; >> + interrupts = <0 68 0x4>; >> + }; >> + >> + counter32k: counter@44e86000 { >> + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; >> + reg = <0x44e86000 0x40>; >> + }; >> + }; >> +}; >> > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 <a-kishore@ti.com> >>> Signed-off-by: Afzal Mohammed <afzal@ti.com> >>> --- >>> >>> 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 0000000..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@48040000 { >>> + compatible = "ti,am4372-timer","ti,am335x-timer"; >>> + reg = <0x48040000 0x400>; >>> + interrupts = <0 68 0x4>; >>> + }; >>> + >>> + counter32k: counter@44e86000 { >>> + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; >>> + reg = <0x44e86000 0x40>; >>> + }; >>> + }; >>> +}; >>> >> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" 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/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi new file mode 100644 index 0000000..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@48040000 { + compatible = "ti,am4372-timer","ti,am335x-timer"; + reg = <0x48040000 0x400>; + interrupts = <0 68 0x4>; + }; + + counter32k: counter@44e86000 { + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; + reg = <0x44e86000 0x40>; + }; + }; +};