2023-01-29 15:21:17

by Jonathan Neuschäfer

Subject: [PATCH] MIPS: dts: lantiq: 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 Lantiq
Danube DT, at /sram@1f000000/eiu@101000.

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

diff --git a/arch/mips/boot/dts/lantiq/danube.dtsi b/arch/mips/boot/dts/lantiq/danube.dtsi
index 510be63c8bdf1..7a7ba66aa5349 100644
--- a/arch/mips/boot/dts/lantiq/danube.dtsi
+++ b/arch/mips/boot/dts/lantiq/danube.dtsi
@@ -40,7 +40,6 @@ sram@1f000000 {
eiu0: eiu@101000 {
#interrupt-cells = <1>;
interrupt-controller;
- interrupt-parent;
compatible = "lantiq,eiu-xway";
reg = <0x101000 0x1000>;
};
--
2.39.0

2023-02-05 16:19:49

by Thomas Bogendoerfer

Subject: Re: [PATCH] MIPS: dts: lantiq: Remove bogus interrupt-parent; line

On Sun, Jan 29, 2023 at 04:20:58PM +0100, Jonathan Neusch?fer wrote:
> Specifying interrupt-parent without a value (in other words, as a bool)
> doesn't really mean anything. Remove one such property in the Lantiq
> Danube DT, at /sram@1f000000/eiu@101000.
>
> Signed-off-by: Jonathan Neusch?fer <[email protected]>
> ---
> arch/mips/boot/dts/lantiq/danube.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/mips/boot/dts/lantiq/danube.dtsi b/arch/mips/boot/dts/lantiq/danube.dtsi
> index 510be63c8bdf1..7a7ba66aa5349 100644
> --- a/arch/mips/boot/dts/lantiq/danube.dtsi
> +++ b/arch/mips/boot/dts/lantiq/danube.dtsi
> @@ -40,7 +40,6 @@ sram@1f000000 {
> eiu0: eiu@101000 {
> #interrupt-cells = <1>;
> interrupt-controller;
> - interrupt-parent;
> compatible = "lantiq,eiu-xway";
> reg = <0x101000 0x1000>;
> };
> --
> 2.39.0

applied to mips-next.

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]