[PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

Subsystems: linux for powerpc (32-bit and 64-bit), the rest

STALE3173d

5 messages, 3 authors, 2017-12-06 · open the first message on its own page

[PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

From: Fabio Estevam <hidden>
Date: 2017-10-17 13:01:32

'interrupts-parent' property does not exist. Fix the typo.

Fixes: 00f3ca740a9c26 ("powerpc/40x: AMCC PowerPC 405EZ Acadia DTS")
Signed-off-by: Fabio Estevam <redacted>
---
 arch/powerpc/boot/dts/acadia.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/acadia.dts b/arch/powerpc/boot/dts/acadia.dts
index 57291f6..8626615 100644
--- a/arch/powerpc/boot/dts/acadia.dts
+++ b/arch/powerpc/boot/dts/acadia.dts
@@ -183,7 +183,7 @@
 			usb@ef603000 {
 				compatible = "ohci-be";
 				reg = <0xef603000 0x80>;
-				interrupts-parent = <&UIC0>;
+				interrupt-parent = <&UIC0>;
 				interrupts = <0xd 0x4 0xe 0x4>;
 			};
 
-- 
2.7.4

Re: [PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

From: Fabio Estevam <festevam@gmail.com>
Date: 2017-12-01 16:22:11

Hi Michael,

On Tue, Oct 17, 2017 at 11:01 AM, Fabio Estevam [off-list ref] wrote:
quoted hunk
'interrupts-parent' property does not exist. Fix the typo.

Fixes: 00f3ca740a9c26 ("powerpc/40x: AMCC PowerPC 405EZ Acadia DTS")
Signed-off-by: Fabio Estevam <redacted>
---
 arch/powerpc/boot/dts/acadia.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/acadia.dts b/arch/powerpc/boot/dts/acadia.dts
index 57291f6..8626615 100644
--- a/arch/powerpc/boot/dts/acadia.dts
+++ b/arch/powerpc/boot/dts/acadia.dts
@@ -183,7 +183,7 @@
                        usb@ef603000 {
                                compatible = "ohci-be";
                                reg = <0xef603000 0x80>;
-                               interrupts-parent = <&UIC0>;
+                               interrupt-parent = <&UIC0>;
Any comments?

Re: [PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2017-12-04 04:45:44

Fabio Estevam [off-list ref] writes:
Hi Michael,

On Tue, Oct 17, 2017 at 11:01 AM, Fabio Estevam [off-list ref] wrote:
quoted
'interrupts-parent' property does not exist. Fix the typo.

Fixes: 00f3ca740a9c26 ("powerpc/40x: AMCC PowerPC 405EZ Acadia DTS")
Signed-off-by: Fabio Estevam <redacted>
---
 arch/powerpc/boot/dts/acadia.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/boot/dts/acadia.dts b/arch/powerpc/boot/dts/acadia.dts
index 57291f6..8626615 100644
--- a/arch/powerpc/boot/dts/acadia.dts
+++ b/arch/powerpc/boot/dts/acadia.dts
@@ -183,7 +183,7 @@
                        usb@ef603000 {
                                compatible = "ohci-be";
                                reg = <0xef603000 0x80>;
-                               interrupts-parent = <&UIC0>;
+                               interrupt-parent = <&UIC0>;
Any comments?
Nope :)

Will pick it up for 4.16.

cheers

Re: [PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

From: Fabio Estevam <festevam@gmail.com>
Date: 2017-12-04 09:16:09

Hi Michael,

On Mon, Dec 4, 2017 at 2:45 AM, Michael Ellerman [off-list ref] wrote:
Will pick it up for 4.16.
Just realized that there is already a fix for this in linux-next:

commit 3d2d4339cc326c427638daa67e264dd455ee1899
Author: Geert Uytterhoeven [off-list ref]
Date:   Fri Jun 2 14:38:47 2017 +0200

    powerpc: dts: acadia: DT fix s/#interrupts-parent/#interrupt-parent/

    Signed-off-by: Geert Uytterhoeven [off-list ref]
    Acked-by: Rob Herring [off-list ref]
    Signed-off-by: Michael Ellerman [off-list ref]

Re: [PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2017-12-06 11:00:11

Fabio Estevam [off-list ref] writes:
Hi Michael,

On Mon, Dec 4, 2017 at 2:45 AM, Michael Ellerman [off-list ref] wrote:
quoted
Will pick it up for 4.16.
Just realized that there is already a fix for this in linux-next:

commit 3d2d4339cc326c427638daa67e264dd455ee1899
Author: Geert Uytterhoeven [off-list ref]
Date:   Fri Jun 2 14:38:47 2017 +0200

    powerpc: dts: acadia: DT fix s/#interrupts-parent/#interrupt-parent/

    Signed-off-by: Geert Uytterhoeven [off-list ref]
    Acked-by: Rob Herring [off-list ref]
    Signed-off-by: Michael Ellerman [off-list ref]

Actually it's in mainline - because I merged it!

OK, I was a bit confused for a while there. But it's just that you and
Geert have both sent a fix for the same issue, and I merged his fix.

Sorry for the confusion.

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help