[PATCH 0/2] arm64: dts: renesas: r8a77990: Revise some nodes

[PATCH 0/2] arm64: dts: renesas: r8a77990: Revise some nodes

[Yoshihiro Shimoda]

This patch set is based on the renesas-devel-20180424-v4.17-rc2 tag
of renesas.git.

This patch set is from https://patchwork.kernel.org/patch/10352549/.
About scif node, I'll submit such a revising patch later (maybe at adding
other scif nodes timing).

Yoshihiro Shimoda (2):
  arm64: dts: renesas: r8a77990: Revise the cache controller node
  arm64: dts: renesas: r8a77990: Revise the psci node

 arch/arm64/boot/dts/renesas/r8a77990.dtsi | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

-- 
1.9.1

[PATCH 1/2] arm64: dts: renesas: r8a77990: Revise the cache controller node

[Yoshihiro Shimoda]

The cache controller node should not have unit-addresses and reg
properties. So, this patch revises them.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77990.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
index 3a19b9e..ba902a2 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
@@ -27,9 +27,8 @@
 			enable-method = "psci";
 		};
 
-		L2_CA53: cache-controller@0 {
+		L2_CA53: cache-controller-0 {
 			compatible = "cache";
-			reg = <0>;
 			power-domains = <&sysc 21>;
 			cache-unified;
 			cache-level = <2>;
-- 
1.9.1

[PATCH 2/2] arm64: dts: renesas: r8a77990: Revise the psci node

[Yoshihiro Shimoda]

The basic support patch 2d2dbadba421 ("arm64: dts: renesas: Add Renesas
R8A77990 SoC support") lacks the compatible "arm,psci-1.0" in the psci
node. So, this patch revises it.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
index ba902a2..4658029 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
@@ -49,7 +49,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci-0.2";
+		compatible = "arm,psci-1.0", "arm,psci-0.2";
 		method = "smc";
 	};
 
-- 
1.9.1

Re: [PATCH 1/2] arm64: dts: renesas: r8a77990: Revise the cache controller node

[Geert Uytterhoeven]

On Wed, Apr 25, 2018 at 10:20 AM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> The cache controller node should not have unit-addresses and reg
> properties. So, this patch revises them.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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 2/2] arm64: dts: renesas: r8a77990: Revise the psci node

[Geert Uytterhoeven]

On Wed, Apr 25, 2018 at 10:20 AM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> The basic support patch 2d2dbadba421 ("arm64: dts: renesas: Add Renesas
> R8A77990 SoC support") lacks the compatible "arm,psci-1.0" in the psci
> node. So, this patch revises it.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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 1/2] arm64: dts: renesas: r8a77990: Revise the cache controller node

[Sergei Shtylyov]

On 04/25/2018 11:20 AM, Yoshihiro Shimoda wrote:

> The cache controller node should not have unit-addresses and reg
> properties. So, this patch revises them.

   Removes?

> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> ---
>  arch/arm64/boot/dts/renesas/r8a77990.dtsi | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> index 3a19b9e..ba902a2 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi
> @@ -27,9 +27,8 @@
>  			enable-method = "psci";
>  		};
>  
> -		L2_CA53: cache-controller@0 {
> +		L2_CA53: cache-controller-0 {
>  			compatible = "cache";
> -			reg = <0>;
>  			power-domains = <&sysc 21>;
>  			cache-unified;
>  			cache-level = <2>;
> 

MBR, Sergei

Re: [PATCH 2/2] arm64: dts: renesas: r8a77990: Revise the psci node

[Simon Horman]

On Wed, Apr 25, 2018 at 11:13:39AM +0200, Geert Uytterhoeven wrote:
> On Wed, Apr 25, 2018 at 10:20 AM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > The basic support patch 2d2dbadba421 ("arm64: dts: renesas: Add Renesas
> > R8A77990 SoC support") lacks the compatible "arm,psci-1.0" in the psci
> > node. So, this patch revises it.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

Re: [PATCH 1/2] arm64: dts: renesas: r8a77990: Revise the cache controller node

[Simon Horman]

On Wed, Apr 25, 2018 at 11:13:23AM +0200, Geert Uytterhoeven wrote:
> On Wed, Apr 25, 2018 at 10:20 AM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > The cache controller node should not have unit-addresses and reg
> > properties. So, this patch revises them.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

On Wed, Apr 25, 2018 at 07:15:31PM +0300, Sergei Shtylyov wrote:
> On 04/25/2018 11:20 AM, Yoshihiro Shimoda wrote:
> 
> > The cache controller node should not have unit-addresses and reg
> > properties. So, this patch revises them.
> 
>    Removes?

Thanks, applied with Geert's tag and Sergei's suggestion.