| Message ID | 1364297952-32473-1-git-send-email-nicolas.ferre@atmel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi, I just saw this since it came in through a pull request On Tue, Mar 26, 2013 at 4:39 AM, Nicolas Ferre <nicolas.ferre@atmel.com> wrote: > From: Douglas Gilbert <dgilbert@interlog.com> > > Signed-off-by: Douglas Gilbert <dgilbert@interlog.com> > Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com> > --- > arch/arm/boot/dts/ariag25.dts | 175 ++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 175 insertions(+) > create mode 100644 arch/arm/boot/dts/ariag25.dts > > diff --git a/arch/arm/boot/dts/ariag25.dts b/arch/arm/boot/dts/ariag25.dts > new file mode 100644 > index 0000000..b43266f > --- /dev/null > +++ b/arch/arm/boot/dts/ariag25.dts Please prefix the boards with the platform. Most other SoCs already do this, and I see now that some at91 boards haven't been prefixed in the past, but it's a good idea to not add more of them. So, at91-ariag25.dts is a good name in this case. -Olof
On 13-04-02 02:48 PM, Olof Johansson wrote: > Hi, > > I just saw this since it came in through a pull request > > On Tue, Mar 26, 2013 at 4:39 AM, Nicolas Ferre <nicolas.ferre@atmel.com> wrote: >> From: Douglas Gilbert <dgilbert@interlog.com> >> >> Signed-off-by: Douglas Gilbert <dgilbert@interlog.com> >> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com> >> --- >> arch/arm/boot/dts/ariag25.dts | 175 ++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 175 insertions(+) >> create mode 100644 arch/arm/boot/dts/ariag25.dts >> >> diff --git a/arch/arm/boot/dts/ariag25.dts b/arch/arm/boot/dts/ariag25.dts >> new file mode 100644 >> index 0000000..b43266f >> --- /dev/null >> +++ b/arch/arm/boot/dts/ariag25.dts > > Please prefix the boards with the platform. Most other SoCs already do > this, and I see now that some at91 boards haven't been prefixed in the > past, but it's a good idea to not add more of them. > > So, at91-ariag25.dts is a good name in this case. That is fine with me. I have changed my working DT config file to at91-ariag25.dts and will push that out to Robert Nelson soon. Also I'm working on a DT version of an existing Acme product called the FoxG20 which is based on the AT91SAM9G20 SoC. It already has non-DT support inside the mainline kernel. I am using at91-foxg20.dts for its DT config file. Doug Gilbert
On 04/03/2013 09:10 PM, Douglas Gilbert : > On 13-04-02 02:48 PM, Olof Johansson wrote: >> Hi, >> >> I just saw this since it came in through a pull request >> >> On Tue, Mar 26, 2013 at 4:39 AM, Nicolas Ferre >> <nicolas.ferre@atmel.com> wrote: >>> From: Douglas Gilbert <dgilbert@interlog.com> >>> >>> Signed-off-by: Douglas Gilbert <dgilbert@interlog.com> >>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com> >>> --- >>> arch/arm/boot/dts/ariag25.dts | 175 >>> ++++++++++++++++++++++++++++++++++++++++++ >>> 1 file changed, 175 insertions(+) >>> create mode 100644 arch/arm/boot/dts/ariag25.dts >>> >>> diff --git a/arch/arm/boot/dts/ariag25.dts >>> b/arch/arm/boot/dts/ariag25.dts >>> new file mode 100644 >>> index 0000000..b43266f >>> --- /dev/null >>> +++ b/arch/arm/boot/dts/ariag25.dts >> >> Please prefix the boards with the platform. Most other SoCs already do >> this, and I see now that some at91 boards haven't been prefixed in the >> past, but it's a good idea to not add more of them. >> >> So, at91-ariag25.dts is a good name in this case. > > That is fine with me. I have changed my working DT > config file to at91-ariag25.dts and will push that > out to Robert Nelson soon. > > Also I'm working on a DT version of an existing Acme > product called the FoxG20 which is based on the > AT91SAM9G20 SoC. It already has non-DT support inside > the mainline kernel. I am using at91-foxg20.dts for > its DT config file. Yep! Works for me. Bye,
diff --git a/arch/arm/boot/dts/ariag25.dts b/arch/arm/boot/dts/ariag25.dts new file mode 100644 index 0000000..b43266f --- /dev/null +++ b/arch/arm/boot/dts/ariag25.dts @@ -0,0 +1,175 @@ +/* + * ariag25.dts - Device Tree file for Acme Systems Aria G25 (AT91SAM9G25 based) + * + * Copyright (C) 2013 Douglas Gilbert <dgilbert@interlog.com>, + * Robert Nelson <robertcnelson@gmail.com> + * + * Licensed under GPLv2 or later. + */ +/dts-v1/; +/include/ "at91sam9g25.dtsi" + +/ { + model = "Acme Systems Aria G25"; + compatible = "acme,ariag25", "atmel,at91sam9x5ek", + "atmel,at91sam9x5", "atmel,at91sam9"; + + aliases { + serial0 = &dbgu; + serial1 = &usart0; + serial2 = &usart1; + serial3 = &usart2; + serial4 = &usart3; + serial5 = &uart0; + }; + + chosen { + bootargs = "console=ttyS0,115200 root=/dev/mmcblk0p2 rw rootwait"; + }; + + memory { + /* 128 MB, change this for 256 MB revision */ + reg = <0x20000000 0x8000000>; + }; + + clocks { + #address-cells = <1>; + #size-cells = <1>; + ranges; + + main_clock: clock@0 { + compatible = "atmel,osc", "fixed-clock"; + clock-frequency = <12000000>; + }; + }; + + ahb { + apb { + mmc0: mmc@f0008000 { + /* N.B. Aria has no SD card detect (CD), assumed present */ + + pinctrl-0 = < + &pinctrl_mmc0_slot0_clk_cmd_dat0 + &pinctrl_mmc0_slot0_dat1_3>; + status = "okay"; + slot@0 { + reg = <0>; + bus-width = <4>; + }; + }; + + i2c0: i2c@f8010000 { + status = "okay"; + }; + + i2c1: i2c@f8014000 { + status = "okay"; + }; + + /* TWD2+TCLK2 hidden behind ethernet, so no i2c2 */ + + usart0: serial@f801c000 { + pinctrl-0 = <&pinctrl_usart0 + &pinctrl_usart0_rts + &pinctrl_usart0_cts>; + status = "okay"; + }; + + usart1: serial@f8020000 { + pinctrl-0 = <&pinctrl_usart1 + /* &pinctrl_usart1_rts */ + /* &pinctrl_usart1_cts */ + >; + status = "okay"; + }; + + usart2: serial@f8024000 { + /* cannot activate RTS2+CTS2, clash with + * ethernet on PB0 and PB1 */ + pinctrl-0 = <&pinctrl_usart2>; + status = "okay"; + }; + + usart3: serial@f8028000 { + compatible = "atmel,at91sam9260-usart"; + reg = <0xf8028000 0x200>; + interrupts = <8 4 5>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_usart3 + /* &pinctrl_usart3_rts */ + /* &pinctrl_usart3_cts */ + >; + status = "okay"; + }; + + macb0: ethernet@f802c000 { + phy-mode = "rmii"; + /* + * following can be overwritten by bootloader: + * for example u-boot 'ftd set' command + */ + local-mac-address = [00 00 00 00 00 00]; + status = "okay"; + }; + + uart0: serial@f8040000 { + compatible = "atmel,at91sam9260-usart"; + reg = <0xf8040000 0x200>; + interrupts = <15 4 5>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_uart0>; + status = "okay"; + }; + + adc0: adc@f804c000 { + status = "okay"; + atmel,adc-channels-used = <0xf>; + atmel,adc-num-channels = <4>; + }; + + dbgu: serial@fffff200 { + status = "okay"; + }; + + pinctrl@fffff400 { + w1_0 { + pinctrl_w1_0: w1_0-0 { + atmel,pins = <0 21 0x0 0x1>; /* PA21 PIO, pull-up */ + }; + }; + }; + + rtc@fffffeb0 { + status = "okay"; + }; + }; + + usb0: ohci@00600000 { + status = "okay"; + num-ports = <3>; + }; + + usb1: ehci@00700000 { + status = "okay"; + }; + }; + + leds { + compatible = "gpio-leds"; + + /* little green LED in middle of Aria G25 module */ + aria_led { + label = "aria_led"; + gpios = <&pioB 8 0>; /* PB8 */ + linux,default-trigger = "heartbeat"; + }; + + }; + + onewire@0 { + compatible = "w1-gpio"; + gpios = <&pioA 21 1>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_w1_0>; + }; +};