* [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups
@ 2014-08-20 14:28 Geert Uytterhoeven
[not found] ` <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
` (5 more replies)
0 siblings, 6 replies; 9+ 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] 9+ messages in thread[parent not found: <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>]
* [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; 9+ 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] 9+ messages in thread
* [PATCH 2/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent 2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven [not found] ` <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> @ 2014-08-20 14:28 ` Geert Uytterhoeven 2014-08-20 14:28 ` [PATCH 3/4] " Geert Uytterhoeven ` (3 subsequent siblings) 5 siblings, 0 replies; 9+ 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] 9+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7790 dtsi: Remove superfluous interrupt-parent 2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven [not found] ` <1408544914-21742-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> 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 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top Geert Uytterhoeven ` (2 subsequent siblings) 5 siblings, 0 replies; 9+ 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] 9+ messages in thread
* [PATCH 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top 2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven ` (2 preceding siblings ...) 2014-08-20 14:28 ` [PATCH 3/4] " 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:32 ` [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven 5 siblings, 0 replies; 9+ 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] 9+ messages in thread
* [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent 2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven ` (3 preceding siblings ...) 2014-08-20 14:28 ` [PATCH 4/4] ARM: shmobile: sh73a0 dtsi: Move interrupt-parent to the top 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 5 siblings, 0 replies; 9+ 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] 9+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups 2014-08-20 14:28 [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven ` (4 preceding siblings ...) 2014-08-20 14:31 ` [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent Geert Uytterhoeven @ 2014-08-20 14:32 ` Geert Uytterhoeven 2014-08-20 14:53 ` Magnus Damm 5 siblings, 1 reply; 9+ 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@vger.kernel.org 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] 9+ messages in thread
* Re: [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups 2014-08-20 14:32 ` [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven @ 2014-08-20 14:53 ` Magnus Damm 2014-08-22 3:39 ` Simon Horman 0 siblings, 1 reply; 9+ 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@vger.kernel.org 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] 9+ 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 0 siblings, 0 replies; 9+ 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@vger.kernel.org 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] 9+ messages in thread
end of thread, other threads:[~2014-08-22 3:39 UTC | newest]
Thread overview: 9+ 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
[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 ` [PATCH 2/4] ARM: shmobile: r8a7790 " Geert Uytterhoeven
2014-08-20 14:28 ` [PATCH 3/4] " 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:31 ` [PATCH 3/4] ARM: shmobile: r8a7791 dtsi: Remove superfluous interrupt-parent Geert Uytterhoeven
2014-08-20 14:32 ` [PATCH 0/4] ARM: shmobile: dtsi: interrupt-parent cleanups Geert Uytterhoeven
2014-08-20 14:53 ` Magnus Damm
2014-08-22 3:39 ` Simon Horman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).