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@xxxxxx> >>> Signed-off-by: Afzal Mohammed <afzal@xxxxxx> >>> --- >>> >>> 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-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html