powerpc: dts: 52xx: drop critical IRQ from dts files

Started by Wolfram Sang on Aug 23, 2012 · 3 messages from 2 authors · Last reply Aug 24, 2012

Subsystems (2)

Maintainers (2)

STALE5192d

open the first message on its own page

Message details and related content
The wakeup-gpios have been assigned a critical IRQ in current
devicetrees. The PIC driver does not support critical IRQs, though, which
leads to the following error when booting (a PCM030 in this case):

	mpc52xx_irqhost_map: invalid irq: virq=16, l1=0, l2=3
	irq: irq-16==>hwirq-0x3 mapping failed: -22
	[WARNing skipped]

Remove these entries until they are supported, if ever.

Signed-off-by: Wolfram Sang <redacted>
Cc: John Bonesio <redacted>
Cc: Anatolij Gustschin <agust@denx.de>
---
 arch/powerpc/boot/dts/lite5200.dts  |    2 +-
 arch/powerpc/boot/dts/mpc5200b.dtsi |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/boot/dts/lite5200.dts b/arch/powerpc/boot/dts/lite5200.dts
index 179a178..25adace 100644
--- a/arch/powerpc/boot/dts/lite5200.dts
+++ b/arch/powerpc/boot/dts/lite5200.dts
@@ -141,7 +141,7 @@
 		gpio@c00 {
 			compatible = "fsl,mpc5200-gpio-wkup";
 			reg = <0xc00 0x40>;
-			interrupts = <1 8 0 0 3 0>;
+			interrupts = <1 8 0>;
 			gpio-controller;
 			#gpio-cells = <2>;
 		};
diff --git a/arch/powerpc/boot/dts/mpc5200b.dtsi b/arch/powerpc/boot/dts/mpc5200b.dtsi
index 7ab286a..e610d68 100644
--- a/arch/powerpc/boot/dts/mpc5200b.dtsi
+++ b/arch/powerpc/boot/dts/mpc5200b.dtsi
@@ -141,7 +141,7 @@
 		gpio_wkup: gpio@c00 {
 			compatible = "fsl,mpc5200b-gpio-wkup","fsl,mpc5200-gpio-wkup";
 			reg = <0xc00 0x40>;
-			interrupts = <1 8 0 0 3 0>;
+			interrupts = <1 8 0>;
 			gpio-controller;
 			#gpio-cells = <2>;
 		};
-- 
1.7.10.4
Message details and related content
On Thu, Aug 23, 2012 at 4:31 PM, Wolfram Sang [off-list ref] wrote:
The wakeup-gpios have been assigned a critical IRQ in current
devicetrees. The PIC driver does not support critical IRQs, though, which
leads to the following error when booting (a PCM030 in this case):

        mpc52xx_irqhost_map: invalid irq: virq=16, l1=0, l2=3
        irq: irq-16==>hwirq-0x3 mapping failed: -22
        [WARNing skipped]

Remove these entries until they are supported, if ever.
The warning is a bug since the irq specifier is valid. Removing the
irqs from the .dts files isn't the solution.

g.
quoted hunk
Signed-off-by: Wolfram Sang <redacted>
Cc: John Bonesio <redacted>
Cc: Anatolij Gustschin <agust@denx.de>
---
 arch/powerpc/boot/dts/lite5200.dts  |    2 +-
 arch/powerpc/boot/dts/mpc5200b.dtsi |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/boot/dts/lite5200.dts b/arch/powerpc/boot/dts/lite5200.dts
index 179a178..25adace 100644
--- a/arch/powerpc/boot/dts/lite5200.dts
+++ b/arch/powerpc/boot/dts/lite5200.dts
@@ -141,7 +141,7 @@
                gpio@c00 {
                        compatible = "fsl,mpc5200-gpio-wkup";
                        reg = <0xc00 0x40>;
-                       interrupts = <1 8 0 0 3 0>;
+                       interrupts = <1 8 0>;
                        gpio-controller;
                        #gpio-cells = <2>;
                };
diff --git a/arch/powerpc/boot/dts/mpc5200b.dtsi b/arch/powerpc/boot/dts/mpc5200b.dtsi
index 7ab286a..e610d68 100644
--- a/arch/powerpc/boot/dts/mpc5200b.dtsi
+++ b/arch/powerpc/boot/dts/mpc5200b.dtsi
@@ -141,7 +141,7 @@
                gpio_wkup: gpio@c00 {
                        compatible = "fsl,mpc5200b-gpio-wkup","fsl,mpc5200-gpio-wkup";
                        reg = <0xc00 0x40>;
-                       interrupts = <1 8 0 0 3 0>;
+                       interrupts = <1 8 0>;
                        gpio-controller;
                        #gpio-cells = <2>;
                };
--
1.7.10.4

_______________________________________________
Linuxppc-dev mailing list

2 more quoted lines



-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
Message details and related content
On Thu, Aug 23, 2012 at 04:59:26PM +0100, Grant Likely wrote:
On Thu, Aug 23, 2012 at 4:31 PM, Wolfram Sang [off-list ref] wrote:
quoted
The wakeup-gpios have been assigned a critical IRQ in current
devicetrees. The PIC driver does not support critical IRQs, though, which
leads to the following error when booting (a PCM030 in this case):

        mpc52xx_irqhost_map: invalid irq: virq=16, l1=0, l2=3
        irq: irq-16==>hwirq-0x3 mapping failed: -22
        [WARNing skipped]

Remove these entries until they are supported, if ever.
The warning is a bug since the irq specifier is valid. Removing the
irqs from the .dts files isn't the solution.
Adding another warning (not error) that critical IRQ are not supported
is?

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help