Re: [PATCH 06/18] ARM: mvebu: a390: add missing compatibility string and add missing bracket

From: Gregory CLEMENT
Date: Wed Jul 27 2016 - 12:14:11 EST


Hi Grzegorz,

On jeu., juil. 21 2016, Grzegorz Jaszczyk <jaz@xxxxxxxxxxxx> wrote:

> The armada-390.dtsi was broken since the first patch which adds Device Tree
> files for Armada 39x SoC was introduced.
>
> Signed-off-by: Grzegorz Jaszczyk <jaz@xxxxxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx> # 4.0+
> Fixes 538da83 ("ARM: mvebu: add Device Tree files for Armada 39x SoC and board")

Change the title prefix to "ARM: dts: mvebu:". Alos usually we use
armada-390 instead of a390 to match the file name.

Acked-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx>

Thanks,

Gregory
> ---
> arch/arm/boot/dts/armada-390.dtsi | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/arm/boot/dts/armada-390.dtsi b/arch/arm/boot/dts/armada-390.dtsi
> index 094e39c..6cd18d8 100644
> --- a/arch/arm/boot/dts/armada-390.dtsi
> +++ b/arch/arm/boot/dts/armada-390.dtsi
> @@ -47,6 +47,8 @@
> #include "armada-39x.dtsi"
>
> / {
> + compatible = "marvell,armada390";
> +
> soc {
> internal-regs {
> pinctrl@18000 {
> @@ -54,4 +56,5 @@
> reg = <0x18000 0x20>;
> };
> };
> + };
> };
> --
> 1.8.3.1
>

--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com