Re: [PATCH v2 1/2] arm64: dts: renesas: Add Renesas R8A77990 SoC support
On Wed, Apr 25, 2018 at 02:47:41AM +, Yoshihiro Shimoda wrote: > Hi, Simon-san, > > > From: Simon Horman, Sent: Tuesday, April 24, 2018 5:37 PM > > > > On Tue, Apr 24, 2018 at 10:26:37AM +0200, Geert Uytterhoeven wrote: > > > Hi Shimoda-san, > > > > > > On Fri, Apr 20, 2018 at 2:28 PM, Yoshihiro Shimoda > > >wrote: > > > > This patch adds basic support for the Renesas R-Car E3 (R8A77990) SoC: > > > > - PSCI > > > > - CPU (single) > > > > - Cache controller > > > > - Main clocks and controller > > > > - Interrupt controller > > > > - Timer > > > > - PMU > > > > - Reset controller > > > > - Product register > > > > - System controller > > > > - UART for console > > > > > > > > Inspried by a patch by Takeshi Kihara in the BSP. > > > > > > > > Signed-off-by: Yoshihiro Shimoda > > > > > > Thanks for you patch! > > > > Thanks for your review. > > > > As I've already applied this patch I'd like to ask Shimoda-san > > to send incremental patches to address the issues you raise below. > > I got it. I'll send incremental patches. Thanks, much appreciated.
RE: [PATCH v2 1/2] arm64: dts: renesas: Add Renesas R8A77990 SoC support
Hi, Simon-san, > From: Simon Horman, Sent: Tuesday, April 24, 2018 5:37 PM > > On Tue, Apr 24, 2018 at 10:26:37AM +0200, Geert Uytterhoeven wrote: > > Hi Shimoda-san, > > > > On Fri, Apr 20, 2018 at 2:28 PM, Yoshihiro Shimoda > >wrote: > > > This patch adds basic support for the Renesas R-Car E3 (R8A77990) SoC: > > > - PSCI > > > - CPU (single) > > > - Cache controller > > > - Main clocks and controller > > > - Interrupt controller > > > - Timer > > > - PMU > > > - Reset controller > > > - Product register > > > - System controller > > > - UART for console > > > > > > Inspried by a patch by Takeshi Kihara in the BSP. > > > > > > Signed-off-by: Yoshihiro Shimoda > > > > Thanks for you patch! > > Thanks for your review. > > As I've already applied this patch I'd like to ask Shimoda-san > to send incremental patches to address the issues you raise below. I got it. I'll send incremental patches. Best regards, Yoshihiro Shimoda > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi > > > @@ -0,0 +1,127 @@ > > > +/* SPDX-License-Identifier: GPL-2.0 */ > > > +/* > > > + * Device Tree Source for the r8a77990 SoC > > > + * > > > + * Copyright (C) 2018 Renesas Electronics Corp. > > > + */ > > > + > > > +#include > > > +#include > > > + > > > +/ { > > > + compatible = "renesas,r8a77990"; > > > + #address-cells = <2>; > > > + #size-cells = <2>; > > > + > > > + cpus { > > > > > + L2_CA53: cache-controller@0 { > > > + compatible = "cache"; > > > + reg = <0>; > > > > Please no unit-addresses and reg properties for cache controllers. > > > > > + power-domains = < 21>; > > > + cache-unified; > > > + cache-level = <2>; > > > + }; > > > + }; > > > > > + psci { > > > + compatible = "arm,psci-0.2"; > > > > "arm,psci-1.0", "arm,psci-0.2"? > > > > > + method = "smc"; > > > + }; > > > + > > > + soc: soc { > > > > > + rst: reset-controller@e616 { > > > + compatible = "renesas,r8a77990-rst"; > > > + reg = <0 0xe616 0 0x0200>; > > > + }; > > > + > > > + sysc: system-controller@e618 { > > > + compatible = "renesas,r8a77990-sysc"; > > > + reg = <0 0xe618 0 0x0400>; > > > + #power-domain-cells = <1>; > > > + }; > > > + > > > + scif2: serial@e6e88000 { > > > + compatible = "renesas,scif-r8a77990", > > > +"renesas,rcar-gen3-scif", > > > "renesas,scif"; > > > + reg = <0 0xe6e88000 0 64>; > > > + interrupts = ; > > > + clocks = < CPG_MOD 310>; > > > + clock-names = "fck"; > > > > I assume you plan to add the other clocks later? That's fine for me. > > > > > + power-domains = < 32>; > > > + resets = < 310>; > > > + status = "disabled"; > > > + }; > > > > Gr{oetje,eeting}s, > > > > Geert > > > > -- > > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > > ge...@linux-m68k.org > > > > In personal conversations with technical people, I call myself a hacker. But > > when I'm talking to journalists I just say "programmer" or something like > > that. > > -- Linus Torvalds > >
Re: [PATCH v2 1/2] arm64: dts: renesas: Add Renesas R8A77990 SoC support
On Tue, Apr 24, 2018 at 10:26:37AM +0200, Geert Uytterhoeven wrote: > Hi Shimoda-san, > > On Fri, Apr 20, 2018 at 2:28 PM, Yoshihiro Shimoda >wrote: > > This patch adds basic support for the Renesas R-Car E3 (R8A77990) SoC: > > - PSCI > > - CPU (single) > > - Cache controller > > - Main clocks and controller > > - Interrupt controller > > - Timer > > - PMU > > - Reset controller > > - Product register > > - System controller > > - UART for console > > > > Inspried by a patch by Takeshi Kihara in the BSP. > > > > Signed-off-by: Yoshihiro Shimoda > > Thanks for you patch! Thanks for your review. As I've already applied this patch I'd like to ask Shimoda-san to send incremental patches to address the issues you raise below. > > --- /dev/null > > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi > > @@ -0,0 +1,127 @@ > > +/* SPDX-License-Identifier: GPL-2.0 */ > > +/* > > + * Device Tree Source for the r8a77990 SoC > > + * > > + * Copyright (C) 2018 Renesas Electronics Corp. > > + */ > > + > > +#include > > +#include > > + > > +/ { > > + compatible = "renesas,r8a77990"; > > + #address-cells = <2>; > > + #size-cells = <2>; > > + > > + cpus { > > > + L2_CA53: cache-controller@0 { > > + compatible = "cache"; > > + reg = <0>; > > Please no unit-addresses and reg properties for cache controllers. > > > + power-domains = < 21>; > > + cache-unified; > > + cache-level = <2>; > > + }; > > + }; > > > + psci { > > + compatible = "arm,psci-0.2"; > > "arm,psci-1.0", "arm,psci-0.2"? > > > + method = "smc"; > > + }; > > + > > + soc: soc { > > > + rst: reset-controller@e616 { > > + compatible = "renesas,r8a77990-rst"; > > + reg = <0 0xe616 0 0x0200>; > > + }; > > + > > + sysc: system-controller@e618 { > > + compatible = "renesas,r8a77990-sysc"; > > + reg = <0 0xe618 0 0x0400>; > > + #power-domain-cells = <1>; > > + }; > > + > > + scif2: serial@e6e88000 { > > + compatible = "renesas,scif-r8a77990", > > +"renesas,rcar-gen3-scif", > > "renesas,scif"; > > + reg = <0 0xe6e88000 0 64>; > > + interrupts = ; > > + clocks = < CPG_MOD 310>; > > + clock-names = "fck"; > > I assume you plan to add the other clocks later? That's fine for me. > > > + power-domains = < 32>; > > + resets = < 310>; > > + status = "disabled"; > > + }; > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > ge...@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like > that. > -- Linus Torvalds >
Re: [PATCH v2 1/2] arm64: dts: renesas: Add Renesas R8A77990 SoC support
Hi Shimoda-san, On Fri, Apr 20, 2018 at 2:28 PM, Yoshihiro Shimodawrote: > This patch adds basic support for the Renesas R-Car E3 (R8A77990) SoC: > - PSCI > - CPU (single) > - Cache controller > - Main clocks and controller > - Interrupt controller > - Timer > - PMU > - Reset controller > - Product register > - System controller > - UART for console > > Inspried by a patch by Takeshi Kihara in the BSP. > > Signed-off-by: Yoshihiro Shimoda Thanks for you patch! > --- /dev/null > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi > @@ -0,0 +1,127 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* > + * Device Tree Source for the r8a77990 SoC > + * > + * Copyright (C) 2018 Renesas Electronics Corp. > + */ > + > +#include > +#include > + > +/ { > + compatible = "renesas,r8a77990"; > + #address-cells = <2>; > + #size-cells = <2>; > + > + cpus { > + L2_CA53: cache-controller@0 { > + compatible = "cache"; > + reg = <0>; Please no unit-addresses and reg properties for cache controllers. > + power-domains = < 21>; > + cache-unified; > + cache-level = <2>; > + }; > + }; > + psci { > + compatible = "arm,psci-0.2"; "arm,psci-1.0", "arm,psci-0.2"? > + method = "smc"; > + }; > + > + soc: soc { > + rst: reset-controller@e616 { > + compatible = "renesas,r8a77990-rst"; > + reg = <0 0xe616 0 0x0200>; > + }; > + > + sysc: system-controller@e618 { > + compatible = "renesas,r8a77990-sysc"; > + reg = <0 0xe618 0 0x0400>; > + #power-domain-cells = <1>; > + }; > + > + scif2: serial@e6e88000 { > + compatible = "renesas,scif-r8a77990", > +"renesas,rcar-gen3-scif", "renesas,scif"; > + reg = <0 0xe6e88000 0 64>; > + interrupts = ; > + clocks = < CPG_MOD 310>; > + clock-names = "fck"; I assume you plan to add the other clocks later? That's fine for me. > + power-domains = < 32>; > + resets = < 310>; > + status = "disabled"; > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds