2022-12-01 10:38:03

by Jonathan Neuschäfer

Subject: [PATCH] ARM: dts: pxa2xx: Remove bogus interrupt-parent; line

Specifying interrupt-parent without a value (in other words, as a bool)
doesn't really mean anything. Remove one such property in the PXA2xx DT,
at /pxabus/interrupt-controller@40d00000.

This patch was tested to cause no detrimental effect on PXA255.

Signed-off-by: Jonathan Neuschäfer <[email protected]>
---
arch/arm/boot/dts/pxa2xx.dtsi | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/pxa2xx.dtsi b/arch/arm/boot/dts/pxa2xx.dtsi
index 1332183f87456..d2c97e1a4478e 100644
--- a/arch/arm/boot/dts/pxa2xx.dtsi
+++ b/arch/arm/boot/dts/pxa2xx.dtsi
@@ -58,7 +58,6 @@ pxairq: interrupt-controller@40d00000 {
#interrupt-cells = <1>;
compatible = "marvell,pxa-intc";
interrupt-controller;
- interrupt-parent;
marvell,intc-nr-irqs = <32>;
reg = <0x40d00000 0xd0>;
};
--
2.35.1

2022-12-01 18:40:55

by Robert Jarzmik

Subject: Re: [PATCH] ARM: dts: pxa2xx: Remove bogus interrupt-parent; line

Jonathan Neuschäfer <[email protected]> writes:

> Specifying interrupt-parent without a value (in other words, as
> a bool)
> doesn't really mean anything. Remove one such property in the
> PXA2xx DT,
> at /pxabus/interrupt-controller@40d00000.
>
> This patch was tested to cause no detrimental effect on PXA255.
>
> Signed-off-by: Jonathan Neuschäfer <[email protected]>
Acked-by: Robert Jarzmik <[email protected]>

--
Robert