linux-kernel - Re: [PATCH 16/16] ARM: at91/dt: Add Acme Arietta G25

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 1 Jun 2015 18:18:46 +0200
From:	Nicolas Ferre <nicolas.ferre@...el.com>
To:	Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
	Sergio Tanzilli <tanzilli@...esystems.it>
CC:	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	Boris Brezillon <boris.brezillon@...e-electrons.com>,
	<linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 16/16] ARM: at91/dt: Add Acme Arietta G25

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 <alexandre.belloni@...e-electrons.com>
> Cc: Sergio Tanzilli <tanzilli@...esystems.it>
> ---
>  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@...08000 {
> +				pinctrl-0 = <
> +				  &pinctrl_mmc0_slot0_clk_cmd_dat0
> +				  &pinctrl_mmc0_slot0_dat1_3>;
> +				status = "okay";
> +
> +				slot@0 {
> +					reg = <0>;
> +					bus-width = <4>;
> +				};
> +			};
> +
> +			usb2: gadget@...3c000 {
> +				status = "okay";
> +			};
> +
> +			dbgu: serial@...ff200 {
> +				status = "okay";
> +			};
> +
> +			rtc@...ffeb0 {
> +				status = "okay";
> +			};
> +		};
> +
> +		usb0: ohci@...00000 {
> +			status = "okay";
> +			num-ports = <3>;
> +		};
> +
> +		usb1: ehci@...00000 {
> +			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 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/