All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-20 14:28 ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

	Hi Simon, Magnus,

This series cleans up interrupt-parent use, to match common use.
It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
thus inherited by all child nodes, and removes superflous "interrupt-parent"
properties from the individual child nodes.

This was tested on r8a7791/koelsch only.

Geert Uytterhoeven (4):
  ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

 arch/arm/boot/dts/r8a7779.dtsi |  6 ------
 arch/arm/boot/dts/r8a7790.dtsi |  1 -
 arch/arm/boot/dts/r8a7791.dtsi |  1 -
 arch/arm/boot/dts/sh73a0.dtsi  | 24 +-----------------------
 4 files changed, 1 insertion(+), 31 deletions(-)

-- 
1.9.1

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] 18+ messages in thread
* [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-20 14:28 ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

	Hi Simon, Magnus,

This series cleans up interrupt-parent use, to match common use.
It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
thus inherited by all child nodes, and removes superflous "interrupt-parent"
properties from the individual child nodes.

This was tested on r8a7791/koelsch only.

Geert Uytterhoeven (4):
  ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

 arch/arm/boot/dts/r8a7779.dtsi |  6 ------
 arch/arm/boot/dts/r8a7790.dtsi |  1 -
 arch/arm/boot/dts/r8a7791.dtsi |  1 -
 arch/arm/boot/dts/sh73a0.dtsi  | 24 +-----------------------
 4 files changed, 1 insertion(+), 31 deletions(-)

-- 
1.9.1

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] 18+ messages in thread
* [PATCH 1/4] ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
       [not found] ` <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
@ 2014-08-20 14:28     ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm
  Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" properties in
the serial nodes can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7779.dtsi | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
index f9dbb7645333..ddc89000854c 100644
--- a/arch/arm/boot/dts/r8a7779.dtsi
+++ b/arch/arm/boot/dts/r8a7779.dtsi
@@ -199,7 +199,6 @@
 	scif0: serial@ffe40000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe40000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -209,7 +208,6 @@
 	scif1: serial@ffe41000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe41000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 89 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -219,7 +217,6 @@
 	scif2: serial@ffe42000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe42000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 90 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -229,7 +226,6 @@
 	scif3: serial@ffe43000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe43000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 91 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -239,7 +235,6 @@
 	scif4: serial@ffe44000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe44000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 92 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -249,7 +244,6 @@
 	scif5: serial@ffe45000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe45000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 93 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 1/4] ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
@ 2014-08-20 14:28     ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm
  Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" properties in
the serial nodes can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
 arch/arm/boot/dts/r8a7779.dtsi | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
index f9dbb7645333..ddc89000854c 100644
--- a/arch/arm/boot/dts/r8a7779.dtsi
+++ b/arch/arm/boot/dts/r8a7779.dtsi
@@ -199,7 +199,6 @@
 	scif0: serial@ffe40000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe40000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -209,7 +208,6 @@
 	scif1: serial@ffe41000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe41000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 89 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -219,7 +217,6 @@
 	scif2: serial@ffe42000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe42000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 90 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -229,7 +226,6 @@
 	scif3: serial@ffe43000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe43000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 91 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -239,7 +235,6 @@
 	scif4: serial@ffe44000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe44000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 92 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
@@ -249,7 +244,6 @@
 	scif5: serial@ffe45000 {
 		compatible = "renesas,scif-r8a7779", "renesas,scif";
 		reg = <0xffe45000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 93 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&cpg_clocks R8A7779_CLK_P>;
 		clock-names = "sci_ick";
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 2/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  2014-08-20 14:28 ` Geert Uytterhoeven
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7790.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index e2e40e5bbe5c..cf1ba907cad9 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -1158,7 +1158,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 2/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7790.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index e2e40e5bbe5c..cf1ba907cad9 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -1158,7 +1158,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
  2014-08-20 14:28 ` Geert Uytterhoeven
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7791.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 152c75ce8534..5620471d35ee 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1156,7 +1156,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7791.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 152c75ce8534..5620471d35ee 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1156,7 +1156,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top
  2014-08-20 14:28 ` Geert Uytterhoeven
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

Add an "interrupt-parent = <&gic>;" at the top, which is inherited by
all child nodes, so the "interrupt-parent" properties can be removed
from the individual child nodes.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/sh73a0.dtsi | 24 +-----------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
index 910b79079d5a..c95935563e44 100644
--- a/arch/arm/boot/dts/sh73a0.dtsi
+++ b/arch/arm/boot/dts/sh73a0.dtsi
@@ -14,6 +14,7 @@
 
 / {
 	compatible = "renesas,sh73a0";
+	interrupt-parent = <&gic>;
 
 	cpus {
 		#address-cells = <1>;
@@ -54,7 +55,6 @@
 			<0xe6900020 1>,
 			<0xe6900040 1>,
 			<0xe6900060 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 1 IRQ_TYPE_LEVEL_HIGH
 			      0 2 IRQ_TYPE_LEVEL_HIGH
 			      0 3 IRQ_TYPE_LEVEL_HIGH
@@ -74,7 +74,6 @@
 			<0xe6900024 1>,
 			<0xe6900044 1>,
 			<0xe6900064 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 9 IRQ_TYPE_LEVEL_HIGH
 			      0 10 IRQ_TYPE_LEVEL_HIGH
 			      0 11 IRQ_TYPE_LEVEL_HIGH
@@ -95,7 +94,6 @@
 			<0xe6900028 1>,
 			<0xe6900048 1>,
 			<0xe6900068 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 17 IRQ_TYPE_LEVEL_HIGH
 			      0 18 IRQ_TYPE_LEVEL_HIGH
 			      0 19 IRQ_TYPE_LEVEL_HIGH
@@ -115,7 +113,6 @@
 			<0xe690002c 1>,
 			<0xe690004c 1>,
 			<0xe690006c 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH
 			      0 26 IRQ_TYPE_LEVEL_HIGH
 			      0 27 IRQ_TYPE_LEVEL_HIGH
@@ -131,7 +128,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6820000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 167 IRQ_TYPE_LEVEL_HIGH
 			      0 168 IRQ_TYPE_LEVEL_HIGH
 			      0 169 IRQ_TYPE_LEVEL_HIGH
@@ -144,7 +140,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6822000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 51 IRQ_TYPE_LEVEL_HIGH
 			      0 52 IRQ_TYPE_LEVEL_HIGH
 			      0 53 IRQ_TYPE_LEVEL_HIGH
@@ -157,7 +152,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6824000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 171 IRQ_TYPE_LEVEL_HIGH
 			      0 172 IRQ_TYPE_LEVEL_HIGH
 			      0 173 IRQ_TYPE_LEVEL_HIGH
@@ -170,7 +164,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6826000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 183 IRQ_TYPE_LEVEL_HIGH
 			      0 184 IRQ_TYPE_LEVEL_HIGH
 			      0 185 IRQ_TYPE_LEVEL_HIGH
@@ -183,7 +176,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6828000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 187 IRQ_TYPE_LEVEL_HIGH
 			      0 188 IRQ_TYPE_LEVEL_HIGH
 			      0 189 IRQ_TYPE_LEVEL_HIGH
@@ -194,7 +186,6 @@
 	mmcif: mmc@e6bd0000 {
 		compatible = "renesas,sh-mmcif";
 		reg = <0xe6bd0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 140 IRQ_TYPE_LEVEL_HIGH
 			      0 141 IRQ_TYPE_LEVEL_HIGH>;
 		reg-io-width = <4>;
@@ -204,7 +195,6 @@
 	sdhi0: sd@ee100000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee100000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 83 IRQ_TYPE_LEVEL_HIGH
 			      0 84 IRQ_TYPE_LEVEL_HIGH
 			      0 85 IRQ_TYPE_LEVEL_HIGH>;
@@ -216,7 +206,6 @@
 	sdhi1: sd@ee120000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee120000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH
 			      0 89 IRQ_TYPE_LEVEL_HIGH>;
 		toshiba,mmc-wrprotect-disable;
@@ -227,7 +216,6 @@
 	sdhi2: sd@ee140000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee140000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 104 IRQ_TYPE_LEVEL_HIGH
 			      0 105 IRQ_TYPE_LEVEL_HIGH>;
 		toshiba,mmc-wrprotect-disable;
@@ -238,7 +226,6 @@
 	scifa0: serial@e6c40000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c40000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 72 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -246,7 +233,6 @@
 	scifa1: serial@e6c50000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c50000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 73 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -254,7 +240,6 @@
 	scifa2: serial@e6c60000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c60000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 74 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -262,7 +247,6 @@
 	scifa3: serial@e6c70000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c70000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 75 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -270,7 +254,6 @@
 	scifa4: serial@e6c80000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c80000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 78 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -278,7 +261,6 @@
 	scifa5: serial@e6cb0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cb0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 79 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -286,7 +268,6 @@
 	scifa6: serial@e6cc0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cc0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -294,7 +275,6 @@
 	scifa7: serial@e6cd0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cd0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 143 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -302,7 +282,6 @@
 	scifb8: serial@e6c30000 {
 		compatible = "renesas,scifb-sh73a0", "renesas,scifb";
 		reg = <0xe6c30000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 80 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -328,7 +307,6 @@
 		#sound-dai-cells = <1>;
 		compatible = "renesas,sh_fsi2";
 		reg = <0xec230000 0x400>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 146 0x4>;
 		status = "disabled";
 	};
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top
@ 2014-08-20 14:28   ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:28 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

Add an "interrupt-parent = <&gic>;" at the top, which is inherited by
all child nodes, so the "interrupt-parent" properties can be removed
from the individual child nodes.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/sh73a0.dtsi | 24 +-----------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
index 910b79079d5a..c95935563e44 100644
--- a/arch/arm/boot/dts/sh73a0.dtsi
+++ b/arch/arm/boot/dts/sh73a0.dtsi
@@ -14,6 +14,7 @@
 
 / {
 	compatible = "renesas,sh73a0";
+	interrupt-parent = <&gic>;
 
 	cpus {
 		#address-cells = <1>;
@@ -54,7 +55,6 @@
 			<0xe6900020 1>,
 			<0xe6900040 1>,
 			<0xe6900060 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 1 IRQ_TYPE_LEVEL_HIGH
 			      0 2 IRQ_TYPE_LEVEL_HIGH
 			      0 3 IRQ_TYPE_LEVEL_HIGH
@@ -74,7 +74,6 @@
 			<0xe6900024 1>,
 			<0xe6900044 1>,
 			<0xe6900064 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 9 IRQ_TYPE_LEVEL_HIGH
 			      0 10 IRQ_TYPE_LEVEL_HIGH
 			      0 11 IRQ_TYPE_LEVEL_HIGH
@@ -95,7 +94,6 @@
 			<0xe6900028 1>,
 			<0xe6900048 1>,
 			<0xe6900068 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 17 IRQ_TYPE_LEVEL_HIGH
 			      0 18 IRQ_TYPE_LEVEL_HIGH
 			      0 19 IRQ_TYPE_LEVEL_HIGH
@@ -115,7 +113,6 @@
 			<0xe690002c 1>,
 			<0xe690004c 1>,
 			<0xe690006c 1>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 25 IRQ_TYPE_LEVEL_HIGH
 			      0 26 IRQ_TYPE_LEVEL_HIGH
 			      0 27 IRQ_TYPE_LEVEL_HIGH
@@ -131,7 +128,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6820000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 167 IRQ_TYPE_LEVEL_HIGH
 			      0 168 IRQ_TYPE_LEVEL_HIGH
 			      0 169 IRQ_TYPE_LEVEL_HIGH
@@ -144,7 +140,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6822000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 51 IRQ_TYPE_LEVEL_HIGH
 			      0 52 IRQ_TYPE_LEVEL_HIGH
 			      0 53 IRQ_TYPE_LEVEL_HIGH
@@ -157,7 +152,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6824000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 171 IRQ_TYPE_LEVEL_HIGH
 			      0 172 IRQ_TYPE_LEVEL_HIGH
 			      0 173 IRQ_TYPE_LEVEL_HIGH
@@ -170,7 +164,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6826000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 183 IRQ_TYPE_LEVEL_HIGH
 			      0 184 IRQ_TYPE_LEVEL_HIGH
 			      0 185 IRQ_TYPE_LEVEL_HIGH
@@ -183,7 +176,6 @@
 		#size-cells = <0>;
 		compatible = "renesas,rmobile-iic";
 		reg = <0xe6828000 0x425>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 187 IRQ_TYPE_LEVEL_HIGH
 			      0 188 IRQ_TYPE_LEVEL_HIGH
 			      0 189 IRQ_TYPE_LEVEL_HIGH
@@ -194,7 +186,6 @@
 	mmcif: mmc@e6bd0000 {
 		compatible = "renesas,sh-mmcif";
 		reg = <0xe6bd0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 140 IRQ_TYPE_LEVEL_HIGH
 			      0 141 IRQ_TYPE_LEVEL_HIGH>;
 		reg-io-width = <4>;
@@ -204,7 +195,6 @@
 	sdhi0: sd@ee100000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee100000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 83 IRQ_TYPE_LEVEL_HIGH
 			      0 84 IRQ_TYPE_LEVEL_HIGH
 			      0 85 IRQ_TYPE_LEVEL_HIGH>;
@@ -216,7 +206,6 @@
 	sdhi1: sd@ee120000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee120000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH
 			      0 89 IRQ_TYPE_LEVEL_HIGH>;
 		toshiba,mmc-wrprotect-disable;
@@ -227,7 +216,6 @@
 	sdhi2: sd@ee140000 {
 		compatible = "renesas,sdhi-sh73a0";
 		reg = <0xee140000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 104 IRQ_TYPE_LEVEL_HIGH
 			      0 105 IRQ_TYPE_LEVEL_HIGH>;
 		toshiba,mmc-wrprotect-disable;
@@ -238,7 +226,6 @@
 	scifa0: serial@e6c40000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c40000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 72 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -246,7 +233,6 @@
 	scifa1: serial@e6c50000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c50000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 73 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -254,7 +240,6 @@
 	scifa2: serial@e6c60000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c60000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 74 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -262,7 +247,6 @@
 	scifa3: serial@e6c70000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c70000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 75 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -270,7 +254,6 @@
 	scifa4: serial@e6c80000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6c80000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 78 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -278,7 +261,6 @@
 	scifa5: serial@e6cb0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cb0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 79 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -286,7 +268,6 @@
 	scifa6: serial@e6cc0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cc0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -294,7 +275,6 @@
 	scifa7: serial@e6cd0000 {
 		compatible = "renesas,scifa-sh73a0", "renesas,scifa";
 		reg = <0xe6cd0000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 143 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -302,7 +282,6 @@
 	scifb8: serial@e6c30000 {
 		compatible = "renesas,scifb-sh73a0", "renesas,scifb";
 		reg = <0xe6c30000 0x100>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 80 IRQ_TYPE_LEVEL_HIGH>;
 		status = "disabled";
 	};
@@ -328,7 +307,6 @@
 		#sound-dai-cells = <1>;
 		compatible = "renesas,sh_fsi2";
 		reg = <0xec230000 0x400>;
-		interrupt-parent = <&gic>;
 		interrupts = <0 146 0x4>;
 		status = "disabled";
 	};
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent
  2014-08-20 14:28 ` Geert Uytterhoeven
@ 2014-08-20 14:31   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:31 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7791.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 152c75ce8534..5620471d35ee 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1156,7 +1156,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent
@ 2014-08-20 14:31   ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:31 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm; +Cc: linux-sh, devicetree, Geert Uytterhoeven

There's already an "interrupt-parent = <&gic>;" at the top, which is
inherited by all child nodes, so the "interrupt-parent" property in
the sound node can be removed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7791.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 152c75ce8534..5620471d35ee 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1156,7 +1156,6 @@
 	rcar_sound: rcar_sound@0xec500000 {
 		#sound-dai-cells = <1>;
 		compatible =  "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2", "renesas,rcar_sound";
-		interrupt-parent = <&gic>;
 		reg =	<0 0xec500000 0 0x1000>, /* SCU */
 			<0 0xec5a0000 0 0x100>,  /* ADG */
 			<0 0xec540000 0 0x1000>, /* SSIU */
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
  2014-08-20 14:28 ` Geert Uytterhoeven
@ 2014-08-20 14:32   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:32 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Simon Horman, Magnus Damm, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
> This series cleans up interrupt-parent use, to match common use.
> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
> thus inherited by all child nodes, and removes superflous "interrupt-parent"
> properties from the individual child nodes.
>
> This was tested on r8a7791/koelsch only.
>
> Geert Uytterhoeven (4):
>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent

Oops, that should have been "r8a7791". I've sent a fixed version of that patch.

>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

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] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-20 14:32   ` Geert Uytterhoeven
  0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2014-08-20 14:32 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Simon Horman, Magnus Damm, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
> This series cleans up interrupt-parent use, to match common use.
> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
> thus inherited by all child nodes, and removes superflous "interrupt-parent"
> properties from the individual child nodes.
>
> This was tested on r8a7791/koelsch only.
>
> Geert Uytterhoeven (4):
>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent

Oops, that should have been "r8a7791". I've sent a fixed version of that patch.

>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

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] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
  2014-08-20 14:32   ` Geert Uytterhoeven
@ 2014-08-20 14:53     ` Magnus Damm
  -1 siblings, 0 replies; 18+ messages in thread
From: Magnus Damm @ 2014-08-20 14:53 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Geert Uytterhoeven, Simon Horman, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 11:32 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
> <geert+renesas@glider.be> wrote:
>> This series cleans up interrupt-parent use, to match common use.
>> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
>> thus inherited by all child nodes, and removes superflous "interrupt-parent"
>> properties from the individual child nodes.
>>
>> This was tested on r8a7791/koelsch only.
>>
>> Geert Uytterhoeven (4):
>>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
>>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>
> Oops, that should have been "r8a7791". I've sent a fixed version of that patch.
>
>>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

Thanks, those look like fine cleanups to me!

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

I suppose both your and my recent patches need a bit of testing, so I
guess it may make sense for Simon to merge one series at a time to
avoid introducing multiple potential issues simultaneously... Combine
early delay issues with Kconfig errors and potential interrupt parent
error and you have something fun to debug. =)

/ magnus

^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-20 14:53     ` Magnus Damm
  0 siblings, 0 replies; 18+ messages in thread
From: Magnus Damm @ 2014-08-20 14:53 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Geert Uytterhoeven, Simon Horman, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 11:32 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
> <geert+renesas@glider.be> wrote:
>> This series cleans up interrupt-parent use, to match common use.
>> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
>> thus inherited by all child nodes, and removes superflous "interrupt-parent"
>> properties from the individual child nodes.
>>
>> This was tested on r8a7791/koelsch only.
>>
>> Geert Uytterhoeven (4):
>>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
>>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
>
> Oops, that should have been "r8a7791". I've sent a fixed version of that patch.
>
>>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top

Thanks, those look like fine cleanups to me!

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

I suppose both your and my recent patches need a bit of testing, so I
guess it may make sense for Simon to merge one series at a time to
avoid introducing multiple potential issues simultaneously... Combine
early delay issues with Kconfig errors and potential interrupt parent
error and you have something fun to debug. =)

/ magnus

^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
  2014-08-20 14:53     ` Magnus Damm
@ 2014-08-22  3:39       ` Simon Horman
  -1 siblings, 0 replies; 18+ messages in thread
From: Simon Horman @ 2014-08-22  3:39 UTC (permalink / raw)
  To: Magnus Damm
  Cc: Geert Uytterhoeven, Geert Uytterhoeven, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 11:53:05PM +0900, Magnus Damm wrote:
> On Wed, Aug 20, 2014 at 11:32 PM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
> > On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
> > <geert+renesas@glider.be> wrote:
> >> This series cleans up interrupt-parent use, to match common use.
> >> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
> >> thus inherited by all child nodes, and removes superflous "interrupt-parent"
> >> properties from the individual child nodes.
> >>
> >> This was tested on r8a7791/koelsch only.
> >>
> >> Geert Uytterhoeven (4):
> >>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
> >>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
> >>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
> >
> > Oops, that should have been "r8a7791". I've sent a fixed version of that patch.
> >
> >>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top
> 
> Thanks, those look like fine cleanups to me!
> 
> Acked-by: Magnus Damm <damm+renesas@opensource.se>

Thanks, applied.

> I suppose both your and my recent patches need a bit of testing, so I
> guess it may make sense for Simon to merge one series at a time to
> avoid introducing multiple potential issues simultaneously... Combine
> early delay issues with Kconfig errors and potential interrupt parent
> error and you have something fun to debug. =)

Hopefully I haven't merged too much :)

^ permalink raw reply	[flat|nested] 18+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-22  3:39       ` Simon Horman
  0 siblings, 0 replies; 18+ messages in thread
From: Simon Horman @ 2014-08-22  3:39 UTC (permalink / raw)
  To: Magnus Damm
  Cc: Geert Uytterhoeven, Geert Uytterhoeven, Linux-sh list, devicetree

On Wed, Aug 20, 2014 at 11:53:05PM +0900, Magnus Damm wrote:
> On Wed, Aug 20, 2014 at 11:32 PM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
> > On Wed, Aug 20, 2014 at 4:28 PM, Geert Uytterhoeven
> > <geert+renesas@glider.be> wrote:
> >> This series cleans up interrupt-parent use, to match common use.
> >> It makes sure there's an "interrupt-parent = <&gic>;" at the top, which is
> >> thus inherited by all child nodes, and removes superflous "interrupt-parent"
> >> properties from the individual child nodes.
> >>
> >> This was tested on r8a7791/koelsch only.
> >>
> >> Geert Uytterhoeven (4):
> >>   ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent
> >>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
> >>   ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent
> >
> > Oops, that should have been "r8a7791". I've sent a fixed version of that patch.
> >
> >>   ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top
> 
> Thanks, those look like fine cleanups to me!
> 
> Acked-by: Magnus Damm <damm+renesas@opensource.se>

Thanks, applied.

> I suppose both your and my recent patches need a bit of testing, so I
> guess it may make sense for Simon to merge one series at a time to
> avoid introducing multiple potential issues simultaneously... Combine
> early delay issues with Kconfig errors and potential interrupt parent
> error and you have something fun to debug. =)

Hopefully I haven't merged too much :)

^ permalink raw reply	[flat|nested] 18+ messages in thread
end of thread, other threads:[~2014-08-22  3:39 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven
2014-08-20 14:28 ` Geert Uytterhoeven
     [not found] ` <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2014-08-20 14:28   ` [PATCH 1/4] ARM: shmobile: r8a7779 dtsi: Remove superfluous interrupt-parent Geert Uytterhoeven
2014-08-20 14:28     ` Geert Uytterhoeven
2014-08-20 14:28 ` [PATCH 2/4] ARM: shmobile: r8a7790 " Geert Uytterhoeven
2014-08-20 14:28   ` Geert Uytterhoeven
2014-08-20 14:28 ` [PATCH 3/4] " Geert Uytterhoeven
2014-08-20 14:28   ` Geert Uytterhoeven
2014-08-20 14:28 ` [PATCH 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top Geert Uytterhoeven
2014-08-20 14:28   ` Geert Uytterhoeven
2014-08-20 14:31 ` [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent Geert Uytterhoeven
2014-08-20 14:31   ` Geert Uytterhoeven
2014-08-20 14:32 ` [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven
2014-08-20 14:32   ` Geert Uytterhoeven
2014-08-20 14:53   ` Magnus Damm
2014-08-20 14:53     ` Magnus Damm
2014-08-22  3:39     ` Simon Horman
2014-08-22  3:39       ` 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.