All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices
@ 2018-04-22 10:08 Magnus Damm
  2018-04-22 10:08 ` [PATCH 01/04] arm64: dts: renesas: r8a7795: " Magnus Damm
                   ` (4 more replies)
  0 siblings, 5 replies; 15+ messages in thread
From: Magnus Damm @ 2018-04-22 10:08 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: horms+renesas, Magnus Damm

arm64: dts: renesas: Enable IPMMU devices

[PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
[PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
[PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
[PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices

Following the policy of using DT to describe the hardware and not software
support state, this series makes sure all IPMMU devices are enabled in DT
for SoCs such as r8a7795, r8a7796, r8a77970 and r8a77995. 

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 Developed on top of renesas-devel-20180420-v4.17-rc1

 arch/arm64/boot/dts/renesas/r8a7795.dtsi  |   12 ------------
 arch/arm64/boot/dts/renesas/r8a7796.dtsi  |    5 -----
 arch/arm64/boot/dts/renesas/r8a77970.dtsi |    2 --
 arch/arm64/boot/dts/renesas/r8a77995.dtsi |   10 ----------
 4 files changed, 29 deletions(-)

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
  2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
@ 2018-04-22 10:08 ` Magnus Damm
  2018-04-23  7:50   ` Geert Uytterhoeven
  2018-04-22 10:08 ` [PATCH 02/04] arm64: dts: renesas: r8a7796: " Magnus Damm
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 15+ messages in thread
From: Magnus Damm @ 2018-04-22 10:08 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: horms+renesas, Magnus Damm

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a7795 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/arm64/boot/dts/renesas/r8a7795.dtsi |   12 ------------
 1 file changed, 12 deletions(-)

--- 0001/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ work/arch/arm64/boot/dts/renesas/r8a7795.dtsi	2018-04-22 18:18:49.150607110 +0900
@@ -947,7 +947,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 2>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ir: mmu@ff8b0000 {
@@ -956,7 +955,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 3>;
 			power-domains = <&sysc R8A7795_PD_A3IR>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mm: mmu@e67b0000 {
@@ -974,7 +972,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 4>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv0: mmu@fd800000 {
@@ -983,7 +980,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 6>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv1: mmu@fd950000 {
@@ -992,7 +988,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 7>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv2: mmu@fd960000 {
@@ -1001,7 +996,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 8>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv3: mmu@fd970000 {
@@ -1010,7 +1004,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 9>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_rt: mmu@ffc80000 {
@@ -1019,7 +1012,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 10>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vc0: mmu@fe6b0000 {
@@ -1028,7 +1020,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 12>;
 			power-domains = <&sysc R8A7795_PD_A3VC>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vc1: mmu@fe6f0000 {
@@ -1037,7 +1028,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 13>;
 			power-domains = <&sysc R8A7795_PD_A3VC>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vi0: mmu@febd0000 {
@@ -1054,7 +1044,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 15>;
 			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vp0: mmu@fe990000 {
@@ -1063,7 +1052,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 16>;
 			power-domains = <&sysc R8A7795_PD_A3VP>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vp1: mmu@fe980000 {

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
  2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
  2018-04-22 10:08 ` [PATCH 01/04] arm64: dts: renesas: r8a7795: " Magnus Damm
@ 2018-04-22 10:08 ` Magnus Damm
  2018-04-23  7:51   ` Geert Uytterhoeven
  2018-04-22 10:08 ` [PATCH 03/04] arm64: dts: renesas: r8a77970: " Magnus Damm
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 15+ messages in thread
From: Magnus Damm @ 2018-04-22 10:08 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: horms+renesas, Magnus Damm

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a7796 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/arm64/boot/dts/renesas/r8a7796.dtsi |    5 -----
 1 file changed, 5 deletions(-)

--- 0001/arch/arm64/boot/dts/renesas/r8a7796.dtsi
+++ work/arch/arm64/boot/dts/renesas/r8a7796.dtsi	2018-04-22 18:22:15.350607110 +0900
@@ -881,7 +881,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 2>;
 			power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ir: mmu@ff8b0000 {
@@ -890,7 +889,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 3>;
 			power-domains = <&sysc R8A7796_PD_A3IR>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mm: mmu@e67b0000 {
@@ -924,7 +922,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 6>;
 			power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_rt: mmu@ffc80000 {
@@ -933,7 +930,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 7>;
 			power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vc0: mmu@fe6b0000 {
@@ -942,7 +938,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 8>;
 			power-domains = <&sysc R8A7796_PD_A3VC>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vi0: mmu@febd0000 {

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
  2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
  2018-04-22 10:08 ` [PATCH 01/04] arm64: dts: renesas: r8a7795: " Magnus Damm
  2018-04-22 10:08 ` [PATCH 02/04] arm64: dts: renesas: r8a7796: " Magnus Damm
@ 2018-04-22 10:08 ` Magnus Damm
  2018-04-23  7:52   ` Geert Uytterhoeven
  2018-04-22 10:08 ` [PATCH 04/04] arm64: dts: renesas: r8a77995: " Magnus Damm
  2018-04-23  7:56 ` [PATCH 00/04] arm64: dts: renesas: " Geert Uytterhoeven
  4 siblings, 1 reply; 15+ messages in thread
From: Magnus Damm @ 2018-04-22 10:08 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: horms+renesas, Magnus Damm

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a77970 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/arm64/boot/dts/renesas/r8a77970.dtsi |    2 --
 1 file changed, 2 deletions(-)

--- 0001/arch/arm64/boot/dts/renesas/r8a77970.dtsi
+++ work/arch/arm64/boot/dts/renesas/r8a77970.dtsi	2018-04-22 18:23:37.480607110 +0900
@@ -137,7 +137,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 9>;
 			power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ir: mmu@ff8b0000 {
@@ -146,7 +145,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 3>;
 			power-domains = <&sysc R8A77970_PD_A3IR>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_rt: mmu@ffc80000 {

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices
  2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
                   ` (2 preceding siblings ...)
  2018-04-22 10:08 ` [PATCH 03/04] arm64: dts: renesas: r8a77970: " Magnus Damm
@ 2018-04-22 10:08 ` Magnus Damm
  2018-04-23  7:52   ` Geert Uytterhoeven
  2018-04-23  7:56 ` [PATCH 00/04] arm64: dts: renesas: " Geert Uytterhoeven
  4 siblings, 1 reply; 15+ messages in thread
From: Magnus Damm @ 2018-04-22 10:08 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: horms+renesas, Magnus Damm

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a77995 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/arm64/boot/dts/renesas/r8a77995.dtsi |   10 ----------
 1 file changed, 10 deletions(-)

--- 0001/arch/arm64/boot/dts/renesas/r8a77995.dtsi
+++ work/arch/arm64/boot/dts/renesas/r8a77995.dtsi	2018-04-22 18:25:39.340607110 +0900
@@ -108,7 +108,6 @@
 			reg = <0 0xfebd0000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 14>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vp0: mmu@fe990000 {
@@ -116,7 +115,6 @@
 			reg = <0 0xfe990000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 16>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vc0: mmu@fe6b0000 {
@@ -124,7 +122,6 @@
 			reg = <0 0xfe6b0000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 12>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv0: mmu@fd800000 {
@@ -132,7 +129,6 @@
 			reg = <0 0xfd800000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 6>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_hc: mmu@e6570000 {
@@ -140,7 +136,6 @@
 			reg = <0 0xe6570000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 2>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_rt: mmu@ffc80000 {
@@ -148,7 +143,6 @@
 			reg = <0 0xffc80000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 10>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mp: mmu@ec670000 {
@@ -156,7 +150,6 @@
 			reg = <0 0xec670000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 4>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ds0: mmu@e6740000 {
@@ -164,7 +157,6 @@
 			reg = <0 0xe6740000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 0>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ds1: mmu@e7740000 {
@@ -172,7 +164,6 @@
 			reg = <0 0xe7740000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 1>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mm: mmu@e67b0000 {
@@ -181,7 +172,6 @@
 			interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>,
 				     <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
  2018-04-22 10:08 ` [PATCH 01/04] arm64: dts: renesas: r8a7795: " Magnus Damm
@ 2018-04-23  7:50   ` Geert Uytterhoeven
  2018-04-23 10:56     ` Simon Horman
  0 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2018-04-23  7:50 UTC (permalink / raw)
  To: Magnus Damm; +Cc: Linux-Renesas, Simon Horman

On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> in DT on the r8a7795 SoC.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
  2018-04-22 10:08 ` [PATCH 02/04] arm64: dts: renesas: r8a7796: " Magnus Damm
@ 2018-04-23  7:51   ` Geert Uytterhoeven
  2018-04-23 10:56     ` Simon Horman
  0 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2018-04-23  7:51 UTC (permalink / raw)
  To: Magnus Damm; +Cc: Linux-Renesas, Simon Horman

On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> in DT on the r8a7796 SoC.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
  2018-04-22 10:08 ` [PATCH 03/04] arm64: dts: renesas: r8a77970: " Magnus Damm
@ 2018-04-23  7:52   ` Geert Uytterhoeven
  2018-04-23 10:57     ` Simon Horman
  0 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2018-04-23  7:52 UTC (permalink / raw)
  To: Magnus Damm; +Cc: Linux-Renesas, Simon Horman

On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> in DT on the r8a77970 SoC.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices
  2018-04-22 10:08 ` [PATCH 04/04] arm64: dts: renesas: r8a77995: " Magnus Damm
@ 2018-04-23  7:52   ` Geert Uytterhoeven
  2018-04-23 11:00     ` Simon Horman
  0 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2018-04-23  7:52 UTC (permalink / raw)
  To: Magnus Damm; +Cc: Linux-Renesas, Simon Horman

On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> in DT on the r8a77995 SoC.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices
  2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
                   ` (3 preceding siblings ...)
  2018-04-22 10:08 ` [PATCH 04/04] arm64: dts: renesas: r8a77995: " Magnus Damm
@ 2018-04-23  7:56 ` Geert Uytterhoeven
  2018-04-24  9:27   ` Simon Horman
  4 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2018-04-23  7:56 UTC (permalink / raw)
  To: Magnus Damm; +Cc: Linux-Renesas, Simon Horman

Hi Magnus,

On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> arm64: dts: renesas: Enable IPMMU devices
>
> [PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
> [PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
> [PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
> [PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices
>
> Following the policy of using DT to describe the hardware and not software
> support state, ...

Good policy! So "[PATCH 0/2] arm64: dts: renesas: r8a77970: Add SMP Support"
can be applied, too?

> ... this series makes sure all IPMMU devices are enabled in DT
> for SoCs such as r8a7795, r8a7796, r8a77970 and r8a77995.

BTW, do you plan to fix arch/arm64/boot/dts/renesas/r8a7795-es1.dtsi, too?

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
  2018-04-23  7:50   ` Geert Uytterhoeven
@ 2018-04-23 10:56     ` Simon Horman
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Horman @ 2018-04-23 10:56 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus Damm, Linux-Renesas

On Mon, Apr 23, 2018 at 09:50:25AM +0200, Geert Uytterhoeven wrote:
> On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> >
> > Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> > in DT on the r8a7795 SoC.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
  2018-04-23  7:51   ` Geert Uytterhoeven
@ 2018-04-23 10:56     ` Simon Horman
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Horman @ 2018-04-23 10:56 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus Damm, Linux-Renesas

On Mon, Apr 23, 2018 at 09:51:15AM +0200, Geert Uytterhoeven wrote:
> On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> >
> > Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> > in DT on the r8a7796 SoC.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
  2018-04-23  7:52   ` Geert Uytterhoeven
@ 2018-04-23 10:57     ` Simon Horman
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Horman @ 2018-04-23 10:57 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus Damm, Linux-Renesas

On Mon, Apr 23, 2018 at 09:52:02AM +0200, Geert Uytterhoeven wrote:
> On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> >
> > Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> > in DT on the r8a77970 SoC.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, I applied this by hand as the node order was changed by a
patch I queued up earlier today. The result is as follows:

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a77970 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
[simon: rebased]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77970.dtsi b/arch/arm64/boot/dts/renesas/r8a77970.dtsi
index 265ecf56a6f2..2159ddaf6e56 100644
--- a/arch/arm64/boot/dts/renesas/r8a77970.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77970.dtsi
@@ -567,7 +567,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 3>;
 			power-domains = <&sysc R8A77970_PD_A3IR>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mm: mmu@e67b0000 {
@@ -593,7 +592,6 @@
 			renesas,ipmmu-main = <&ipmmu_mm 9>;
 			power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		gic: interrupt-controller@f1010000 {
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: [PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices
  2018-04-23  7:52   ` Geert Uytterhoeven
@ 2018-04-23 11:00     ` Simon Horman
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Horman @ 2018-04-23 11:00 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus Damm, Linux-Renesas

On Mon, Apr 23, 2018 at 09:52:41AM +0200, Geert Uytterhoeven wrote:
> On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> >
> > Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
> > in DT on the r8a77995 SoC.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, I applied this by hand as the node order was updated by
a patch that I applied earlier today. The result is as follows:

From: Magnus Damm <damm+renesas@opensource.se>

Remove 'status = "disabled"' to make sure all IPMMU devices are enabled
in DT on the r8a77995 SoC.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
[simon: rebased]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm64/boot/dts/renesas/r8a77995.dtsi | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
index a97830589b0d..ba98865b0c9b 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
@@ -409,7 +409,6 @@
 			reg = <0 0xe6740000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 0>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_ds1: mmu@e7740000 {
@@ -417,7 +416,6 @@
 			reg = <0 0xe7740000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 1>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_hc: mmu@e6570000 {
@@ -425,7 +423,6 @@
 			reg = <0 0xe6570000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 2>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mm: mmu@e67b0000 {
@@ -434,7 +431,6 @@
 			interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>,
 				     <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_mp: mmu@ec670000 {
@@ -442,7 +438,6 @@
 			reg = <0 0xec670000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 4>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_pv0: mmu@fd800000 {
@@ -450,7 +445,6 @@
 			reg = <0 0xfd800000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 6>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_rt: mmu@ffc80000 {
@@ -458,7 +452,6 @@
 			reg = <0 0xffc80000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 10>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vc0: mmu@fe6b0000 {
@@ -466,7 +459,6 @@
 			reg = <0 0xfe6b0000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 12>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vi0: mmu@febd0000 {
@@ -474,7 +466,6 @@
 			reg = <0 0xfebd0000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 14>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		ipmmu_vp0: mmu@fe990000 {
@@ -482,7 +473,6 @@
 			reg = <0 0xfe990000 0 0x1000>;
 			renesas,ipmmu-main = <&ipmmu_mm 16>;
 			#iommu-cells = <1>;
-			status = "disabled";
 		};
 
 		avb: ethernet@e6800000 {
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices
  2018-04-23  7:56 ` [PATCH 00/04] arm64: dts: renesas: " Geert Uytterhoeven
@ 2018-04-24  9:27   ` Simon Horman
  0 siblings, 0 replies; 15+ messages in thread
From: Simon Horman @ 2018-04-24  9:27 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Magnus Damm, Linux-Renesas

On Mon, Apr 23, 2018 at 09:56:06AM +0200, Geert Uytterhoeven wrote:
> Hi Magnus,
> 
> On Sun, Apr 22, 2018 at 12:08 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > arm64: dts: renesas: Enable IPMMU devices
> >
> > [PATCH 01/04] arm64: dts: renesas: r8a7795: Enable IPMMU devices
> > [PATCH 02/04] arm64: dts: renesas: r8a7796: Enable IPMMU devices
> > [PATCH 03/04] arm64: dts: renesas: r8a77970: Enable IPMMU devices
> > [PATCH 04/04] arm64: dts: renesas: r8a77995: Enable IPMMU devices
> >
> > Following the policy of using DT to describe the hardware and not software
> > support state, ...
> 
> Good policy! So "[PATCH 0/2] arm64: dts: renesas: r8a77970: Add SMP Support"
> can be applied, too?
> 
> > ... this series makes sure all IPMMU devices are enabled in DT
> > for SoCs such as r8a7795, r8a7796, r8a77970 and r8a77995.
> 
> BTW, do you plan to fix arch/arm64/boot/dts/renesas/r8a7795-es1.dtsi, too?

I took the liberty of posting a patch to do that.

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2018-04-24  9:27 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-22 10:08 [PATCH 00/04] arm64: dts: renesas: Enable IPMMU devices Magnus Damm
2018-04-22 10:08 ` [PATCH 01/04] arm64: dts: renesas: r8a7795: " Magnus Damm
2018-04-23  7:50   ` Geert Uytterhoeven
2018-04-23 10:56     ` Simon Horman
2018-04-22 10:08 ` [PATCH 02/04] arm64: dts: renesas: r8a7796: " Magnus Damm
2018-04-23  7:51   ` Geert Uytterhoeven
2018-04-23 10:56     ` Simon Horman
2018-04-22 10:08 ` [PATCH 03/04] arm64: dts: renesas: r8a77970: " Magnus Damm
2018-04-23  7:52   ` Geert Uytterhoeven
2018-04-23 10:57     ` Simon Horman
2018-04-22 10:08 ` [PATCH 04/04] arm64: dts: renesas: r8a77995: " Magnus Damm
2018-04-23  7:52   ` Geert Uytterhoeven
2018-04-23 11:00     ` Simon Horman
2018-04-23  7:56 ` [PATCH 00/04] arm64: dts: renesas: " Geert Uytterhoeven
2018-04-24  9:27   ` Simon Horman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.