Re: [PATCH v2 1/2] arm64: dts: renesas: Add CPU topology on R-Car Gen3 SoCs

[Simon Horman]

On Wed, Nov 14, 2018 at 10:50:03AM +0100, Geert Uytterhoeven wrote:
> Hi Inami-san,
> 
> On Thu, Nov 8, 2018 at 8:25 AM Gaku Inami  wrote:
> > This patch adds the "cpu-map" into r8a7795/r8a7796 composed of
> > multi-cluster. This definition is used to parse the cpu topology.
> >
> > Signed-off-by: Gaku Inami 
> 
> Thanks for your patch!
> 
> Next time, please collect tags provided by reviewers on the previous
> version.
> 
> Reviewed-by: Geert Uytterhoeven 

Thanks, applied for v4.21.

RE: [PATCH v2 1/2] arm64: dts: renesas: Add CPU topology on R-Car Gen3 SoCs

[Gaku Inami]

Hi Geert-san,

Thanks for your review.

> -Original Message-
> From: Geert Uytterhoeven 
> Sent: Wednesday, November 14, 2018 6:50 PM
> To: Gaku Inami 
> Cc: Simon Horman ; Magnus Damm ; 
> Rob Herring ; Mark
> Rutland ; Linux-Renesas 
> ; open list:OPEN FIRMWARE AND FLATTENED
> DEVICE TREE BINDINGS 
> Subject: Re: [PATCH v2 1/2] arm64: dts: renesas: Add CPU topology on R-Car 
> Gen3 SoCs

[snip]

> Next time, please collect tags provided by reviewers on the previous
> version.

I am sorry that "reviewed-by" from you is lacked. I will add correct tags next 
time.

Regards,
Inami

Re: [PATCH v2 1/2] arm64: dts: renesas: Add CPU topology on R-Car Gen3 SoCs

[Geert Uytterhoeven]

Hi Inami-san,

On Thu, Nov 8, 2018 at 8:25 AM Gaku Inami  wrote:
> This patch adds the "cpu-map" into r8a7795/r8a7796 composed of
> multi-cluster. This definition is used to parse the cpu topology.
>
> Signed-off-by: Gaku Inami 

Thanks for your patch!

Next time, please collect tags provided by reviewers on the previous
version.

Reviewed-by: Geert Uytterhoeven 

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

[PATCH v2 1/2] arm64: dts: renesas: Add CPU topology on R-Car Gen3 SoCs

[Gaku Inami]

This patch adds the "cpu-map" into r8a7795/r8a7796 composed of
multi-cluster. This definition is used to parse the cpu topology.

Signed-off-by: Gaku Inami 
---
v1 -> v2:
 - Consolidate two patches for r8a7795 and r8a7796 into one patch
---
 arch/arm64/boot/dts/renesas/r8a7795.dtsi | 32 
 arch/arm64/boot/dts/renesas/r8a7796.dtsi | 26 ++
 2 files changed, 58 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi 
b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 0b54c53..63d5b61 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -116,6 +116,38 @@
#address-cells = <1>;
#size-cells = <0>;
 
+   cpu-map {
+   cluster0 {
+   core0 {
+   cpu = <_0>;
+   };
+   core1 {
+   cpu = <_1>;
+   };
+   core2 {
+   cpu = <_2>;
+   };
+   core3 {
+   cpu = <_3>;
+   };
+   };
+
+   cluster1 {
+   core0 {
+   cpu = <_0>;
+   };
+   core1 {
+   cpu = <_1>;
+   };
+   core2 {
+   cpu = <_2>;
+   };
+   core3 {
+   cpu = <_3>;
+   };
+   };
+   };
+
a57_0: cpu@0 {
compatible = "arm,cortex-a57", "arm,armv8";
reg = <0x0>;
diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi 
b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
index 3baee26..b12bf73 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
@@ -127,6 +127,32 @@
#address-cells = <1>;
#size-cells = <0>;
 
+   cpu-map {
+   cluster0 {
+   core0 {
+   cpu = <_0>;
+   };
+   core1 {
+   cpu = <_1>;
+   };
+   };
+
+   cluster1 {
+   core0 {
+   cpu = <_0>;
+   };
+   core1 {
+   cpu = <_1>;
+   };
+   core2 {
+   cpu = <_2>;
+   };
+   core3 {
+   cpu = <_3>;
+   };
+   };
+   };
+
a57_0: cpu@0 {
compatible = "arm,cortex-a57", "arm,armv8";
reg = <0x0>;
-- 
2.7.4