[PATCH 05/11] arm64: dts: renesas: initial R8A77970 SoC device tree

[Sergei Shtylyov]

On 09/13/2017 11:59 AM, Geert Uytterhoeven wrote:

>> The initial R8A77970 SoC device tree including Cortex-A53 CPU, GIC, timer,
>> CPG, RST, and SYSC.
>>
>> Based on the original (and large) patch by Daisuke Matsushita
>> <daisuke.matsushita.ns at hitachi.com>.
>>
>> Signed-off-by: Vladimir Barinov <vladimir.barinov at cogentembedded.com>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov at cogentembedded.com>
>>
>> ---
>>   arch/arm64/boot/dts/renesas/r8a77970.dtsi |  126 ++++++++++++++++++++++++++++++
>>   1 file changed, 126 insertions(+)
>>
>> Index: renesas/arch/arm64/boot/dts/renesas/r8a77970.dtsi
>> ===================================================================
>> --- /dev/null
>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970.dtsi
>> @@ -0,0 +1,126 @@
>> +/*
>> + * Device Tree Source for the r8a77970 SoC
>> + *
>> + * Copyright (C) 2016-2017 Renesas Electronics Corp.
>> + * Copyright (C) 2017 Cogent Embedded, Inc.
>> + *
>> + * 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 <dt-bindings/interrupt-controller/irq.h>
>> +#include <dt-bindings/interrupt-controller/arm-gic.h>
> 
> arm-gic.h includes irq.h.

    I prefer to explicitly #include what's needed, so I'd leave this alone if 
you don't mind...

>> +#include <dt-bindings/clock/renesas-cpg-mssr.h>
>> +#include <dt-bindings/power/r8a77970-sysc.h>
> 
> You can avoid the dependency on the above header, which will go upstream
> through a different branch, by hardcoding the power area indices for the
> initial submission, like you already did for the CPG core clocks.

    OK.

>> +
>> +/ {
>> +       compatible = "renesas,r8a77970";
>> +       #address-cells = <2>;
>> +       #size-cells = <2>;
>> +
>> +       psci {
>> +               compatible = "arm,psci-1.0";
> 
> PSCI 1.0 is backwards compatible with PSI 0.2:
> 
>      compatible = "arm,psci-1.0", "arm,psci-0.2";

    OK.

> Reviewed-by: Geert Uytterhoeven <geert+renesas at glider.be>
> 
> Gr{oetje,eeting}s,
> 
>                          Geert

MBR, Sergei