lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 29 May 2013 10:57:18 +0200 From: Florian Vaussard <florian.vaussard@...l.ch> To: Benoit Cousson <b-cousson@...com> CC: Afzal Mohammed <afzal@...com>, linux-omap@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, devicetree-discuss@...ts.ozlabs.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org, Russell King <linux@....linux.org.uk>, Rob Herring <rob.herring@...xeda.com>, Ankur Kishore <a-kishore@...com>, Grant Likely <grant.likely@...aro.org>, Benoit Cousson <benoit.cousson@...aro.org> Subject: 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 <a-kishore@...com> >> Signed-off-by: Afzal Mohammed <afzal@...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@...41000 { >> + 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@...09000 { >> + compatible = "ti,am4372-uart","ti,omap2-uart"; >> + reg = <0x44e09000 0x2000>; >> + interrupts = <0 72 0x4>; >> + }; >> + >> + timer1: timer@...31000 { >> + compatible = "ti,am4372-timer-1ms","ti,am335x-timer-1ms"; >> + reg = <0x44e31000 0x400>; >> + interrupts = <0 67 0x4>; >> + ti,timer-alwon; >> + }; >> + >> + timer2: timer@...40000 { >> + compatible = "ti,am4372-timer","ti,am335x-timer"; >> + reg = <0x48040000 0x400>; >> + interrupts = <0 68 0x4>; >> + }; >> + >> + counter32k: counter@...86000 { >> + compatible = "ti,am4372-counter32k","ti,omap-counter32k"; >> + reg = <0x44e86000 0x40>; >> + }; >> + }; >> +}; >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists