Le 29/05/2015 19:47, Alexandre Belloni a écrit : > Add a minimum Device Tree for Acme Arietta G25. > http://acme.systems/arietta > > Signed-off-by: Alexandre Belloni <[email protected]> > Cc: Sergio Tanzilli <[email protected]> > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/at91-acme-arietta.dts | 75 > +++++++++++++++++++++++++++++++++ > 2 files changed, 76 insertions(+) > create mode 100644 arch/arm/boot/dts/at91-acme-arietta.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 8c3da289a00b..96a169e6ea56 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -31,6 +31,7 @@ dtb-$(CONFIG_SOC_SAM_V4_V5) += \ > at91sam9n12ek.dtb \ > at91sam9rlek.dtb \ > at91-ariag25.dtb \ > + at91-acme-arietta.dtb \
Do you want to have the additional vendor name here? We already have 2 boards by ACME and they don't have this type of name: at91-foxg20.dtb at91-ariag25.dtb So I'd prefer to use "at91-arietta.dtb" or "at91-ariettag25.dtb" or any kind or variant from these ones... I know it may come from the ACME DT generator tool, but it could be strange at first sight. Sergio, do you have an opinion? In relation with eventual future products, which is your preferred name? > at91-cosino_mega2560.dtb \ > at91-kizboxmini.dtb \ > at91sam9g15ek.dtb \ > diff --git a/arch/arm/boot/dts/at91-acme-arietta.dts > b/arch/arm/boot/dts/at91-acme-arietta.dts > new file mode 100644 > index 000000000000..142d588ad6bf > --- /dev/null > +++ b/arch/arm/boot/dts/at91-acme-arietta.dts > @@ -0,0 +1,75 @@ > +/* > + * Device Tree file for Arietta G25 > + * This device tree is minimal, to activate more peripherals, see: > + * http://dts.acmesystems.it/arietta/ Yes, absolutely: link to the DT generator is a big plus here. > + */ > +/dts-v1/; > +#include "at91sam9g25.dtsi" > +/ { > + model = "Acme Systems Arietta G25"; > + compatible = "acme,ariettag25", "atmel,at91sam9x5", "atmel,at91sam9"; > + Don't we need to add a "alias" group here, to be sure that the "stdout-path" is always good whatever appends to the underlying at91sam9g25.dtsi (nitpicking, I know ;-)) ? > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > + > + memory { > + reg = <0x20000000 0x8000000>; > + }; > + > + clocks { > + slow_xtal { > + clock-frequency = <32768>; > + }; > + > + main_xtal { > + clock-frequency = <12000000>; > + }; > + }; > + > + ahb { > + apb { > + mmc0: mmc@f0008000 { > + pinctrl-0 = < > + &pinctrl_mmc0_slot0_clk_cmd_dat0 > + &pinctrl_mmc0_slot0_dat1_3>; > + status = "okay"; > + > + slot@0 { > + reg = <0>; > + bus-width = <4>; > + }; > + }; > + > + usb2: gadget@f803c000 { > + status = "okay"; > + }; > + > + dbgu: serial@fffff200 { > + status = "okay"; > + }; > + > + rtc@fffffeb0 { > + status = "okay"; > + }; > + }; > + > + usb0: ohci@00600000 { > + status = "okay"; > + num-ports = <3>; > + }; > + > + usb1: ehci@00700000 { > + status = "okay"; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + arietta_led { > + label = "arietta_led"; > + gpios = <&pioB 8 GPIO_ACTIVE_HIGH>; /* PB8 */ > + linux,default-trigger = "heartbeat"; > + }; > + }; > +}; > -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
