[PATCH] ARM: dts: OMAP: Move interrupt-parent to the root node to avoid duplication
Hiremath, Vaibhav
hvaibhav at ti.com
Thu Oct 25 04:21:52 EST 2012
On Wed, Oct 24, 2012 at 15:32:14, Cousson, Benoit wrote:
> The interrupt-parent attribute does not have to be added in each
> node since the DT framework will check for the parent as well to get it.
>
> Create an interrupt-parent for OMAP2, OMAP3, AM33xx and remove the
> attributes from every nodes that were using it.
>
> Signed-off-by: Benoit Cousson <b-cousson at ti.com>
> Cc: Vaibhav Hiremath <hvaibhav at ti.com>
> Cc: Peter Ujfalusi <peter.ujfalusi at ti.com>
> Cc: Sebastien Guiriec <s-guiriec at ti.com>
> ---
>
> This patch was triggered by the thread about Seb's patch [1].
> So let's clean the current OMAP/AM stuff right now to be consistent.
>
> The patch applies on top of the for_3.8/dts [2] branch.
>
> Regards,
> Benoit
>
> [1] https://lkml.org/lkml/2012/10/24/85
> [2] git://git.kernel.org/pub/scm/linux/kernel/git/bcousson/linux-omap-dt.git
>
>
> arch/arm/boot/dts/am33xx.dtsi | 17 +----------------
> arch/arm/boot/dts/omap2.dtsi | 1 +
> arch/arm/boot/dts/omap2420.dtsi | 2 --
> arch/arm/boot/dts/omap2430.dtsi | 5 -----
> arch/arm/boot/dts/omap3.dtsi | 6 +-----
> arch/arm/boot/dts/omap4.dtsi | 6 ------
> arch/arm/boot/dts/omap5.dtsi | 5 -----
> 7 files changed, 3 insertions(+), 39 deletions(-)
>
Looks good to me.
Acked-by: Vaibhav Hiremath <hvaibhav at ti.com>
Thanks,
Vaibhav
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index 64c2efe..4709269 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -12,6 +12,7 @@
>
> / {
> compatible = "ti,am33xx";
> + interrupt-parent = <&intc>;
>
> aliases {
> serial0 = &uart1;
> @@ -94,7 +95,6 @@
> interrupt-controller;
> #interrupt-cells = <1>;
> reg = <0x44e07000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <96>;
> };
>
> @@ -106,7 +106,6 @@
> interrupt-controller;
> #interrupt-cells = <1>;
> reg = <0x4804c000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <98>;
> };
>
> @@ -118,7 +117,6 @@
> interrupt-controller;
> #interrupt-cells = <1>;
> reg = <0x481ac000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <32>;
> };
>
> @@ -130,7 +128,6 @@
> interrupt-controller;
> #interrupt-cells = <1>;
> reg = <0x481ae000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <62>;
> };
>
> @@ -139,7 +136,6 @@
> ti,hwmods = "uart1";
> clock-frequency = <48000000>;
> reg = <0x44e09000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <72>;
> status = "disabled";
> };
> @@ -149,7 +145,6 @@
> ti,hwmods = "uart2";
> clock-frequency = <48000000>;
> reg = <0x48022000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <73>;
> status = "disabled";
> };
> @@ -159,7 +154,6 @@
> ti,hwmods = "uart3";
> clock-frequency = <48000000>;
> reg = <0x48024000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <74>;
> status = "disabled";
> };
> @@ -169,7 +163,6 @@
> ti,hwmods = "uart4";
> clock-frequency = <48000000>;
> reg = <0x481a6000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <44>;
> status = "disabled";
> };
> @@ -179,7 +172,6 @@
> ti,hwmods = "uart5";
> clock-frequency = <48000000>;
> reg = <0x481a8000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <45>;
> status = "disabled";
> };
> @@ -189,7 +181,6 @@
> ti,hwmods = "uart6";
> clock-frequency = <48000000>;
> reg = <0x481aa000 0x2000>;
> - interrupt-parent = <&intc>;
> interrupts = <46>;
> status = "disabled";
> };
> @@ -200,7 +191,6 @@
> #size-cells = <0>;
> ti,hwmods = "i2c1";
> reg = <0x44e0b000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <70>;
> status = "disabled";
> };
> @@ -211,7 +201,6 @@
> #size-cells = <0>;
> ti,hwmods = "i2c2";
> reg = <0x4802a000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <71>;
> status = "disabled";
> };
> @@ -222,7 +211,6 @@
> #size-cells = <0>;
> ti,hwmods = "i2c3";
> reg = <0x4819c000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <30>;
> status = "disabled";
> };
> @@ -231,7 +219,6 @@
> compatible = "ti,omap3-wdt";
> ti,hwmods = "wd_timer2";
> reg = <0x44e35000 0x1000>;
> - interrupt-parent = <&intc>;
> interrupts = <91>;
> };
>
> @@ -240,7 +227,6 @@
> ti,hwmods = "d_can0";
> reg = <0x481cc000 0x2000>;
> interrupts = <52>;
> - interrupt-parent = <&intc>;
> status = "disabled";
> };
>
> @@ -249,7 +235,6 @@
> ti,hwmods = "d_can1";
> reg = <0x481d0000 0x2000>;
> interrupts = <55>;
> - interrupt-parent = <&intc>;
> status = "disabled";
> };
> };
> diff --git a/arch/arm/boot/dts/omap2.dtsi b/arch/arm/boot/dts/omap2.dtsi
> index 581cb08..f366482 100644
> --- a/arch/arm/boot/dts/omap2.dtsi
> +++ b/arch/arm/boot/dts/omap2.dtsi
> @@ -12,6 +12,7 @@
>
> / {
> compatible = "ti,omap2430", "ti,omap2420", "ti,omap2";
> + interrupt-parent = <&intc>;
>
> aliases {
> serial0 = &uart1;
> diff --git a/arch/arm/boot/dts/omap2420.dtsi b/arch/arm/boot/dts/omap2420.dtsi
> index bfd76b4..4d5ce91 100644
> --- a/arch/arm/boot/dts/omap2420.dtsi
> +++ b/arch/arm/boot/dts/omap2420.dtsi
> @@ -30,7 +30,6 @@
> interrupts = <59>, /* TX interrupt */
> <60>; /* RX interrupt */
> interrupt-names = "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,hwmods = "mcbsp1";
> };
>
> @@ -41,7 +40,6 @@
> interrupts = <62>, /* TX interrupt */
> <63>; /* RX interrupt */
> interrupt-names = "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,hwmods = "mcbsp2";
> };
> };
> diff --git a/arch/arm/boot/dts/omap2430.dtsi b/arch/arm/boot/dts/omap2430.dtsi
> index 4565d97..fa84532 100644
> --- a/arch/arm/boot/dts/omap2430.dtsi
> +++ b/arch/arm/boot/dts/omap2430.dtsi
> @@ -32,7 +32,6 @@
> <60>, /* RX interrupt */
> <61>; /* RX overflow interrupt */
> interrupt-names = "common", "tx", "rx", "rx_overflow";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp1";
> };
> @@ -45,7 +44,6 @@
> <62>, /* TX interrupt */
> <63>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp2";
> };
> @@ -58,7 +56,6 @@
> <89>, /* TX interrupt */
> <90>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp3";
> };
> @@ -71,7 +68,6 @@
> <54>, /* TX interrupt */
> <55>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp4";
> };
> @@ -84,7 +80,6 @@
> <81>, /* TX interrupt */
> <82>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp5";
> };
> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
> index f38ea87..955cbdc 100644
> --- a/arch/arm/boot/dts/omap3.dtsi
> +++ b/arch/arm/boot/dts/omap3.dtsi
> @@ -12,6 +12,7 @@
>
> / {
> compatible = "ti,omap3430", "ti,omap3";
> + interrupt-parent = <&intc>;
>
> aliases {
> serial0 = &uart1;
> @@ -240,7 +241,6 @@
> <59>, /* TX interrupt */
> <60>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp1";
> };
> @@ -255,7 +255,6 @@
> <63>, /* RX interrupt */
> <4>; /* Sidetone */
> interrupt-names = "common", "tx", "rx", "sidetone";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <1280>;
> ti,hwmods = "mcbsp2";
> };
> @@ -270,7 +269,6 @@
> <90>, /* RX interrupt */
> <5>; /* Sidetone */
> interrupt-names = "common", "tx", "rx", "sidetone";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp3";
> };
> @@ -283,7 +281,6 @@
> <54>, /* TX interrupt */
> <55>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp4";
> };
> @@ -296,7 +293,6 @@
> <81>, /* TX interrupt */
> <82>; /* RX interrupt */
> interrupt-names = "common", "tx", "rx";
> - interrupt-parent = <&intc>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp5";
> };
> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
> index 812461e..2ab6e68 100644
> --- a/arch/arm/boot/dts/omap4.dtsi
> +++ b/arch/arm/boot/dts/omap4.dtsi
> @@ -340,7 +340,6 @@
> <0x49032000 0x7f>; /* L3 Interconnect */
> reg-names = "mpu", "dma";
> interrupts = <0 112 0x4>;
> - interrupt-parent = <&gic>;
> ti,hwmods = "mcpdm";
> };
>
> @@ -350,7 +349,6 @@
> <0x4902e000 0x7f>; /* L3 Interconnect */
> reg-names = "mpu", "dma";
> interrupts = <0 114 0x4>;
> - interrupt-parent = <&gic>;
> ti,hwmods = "dmic";
> };
>
> @@ -361,7 +359,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 17 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp1";
> };
> @@ -373,7 +370,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 22 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp2";
> };
> @@ -385,7 +381,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 23 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp3";
> };
> @@ -396,7 +391,6 @@
> reg-names = "mpu";
> interrupts = <0 16 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp4";
> };
> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> index 42c78be..61a4f2e 100644
> --- a/arch/arm/boot/dts/omap5.dtsi
> +++ b/arch/arm/boot/dts/omap5.dtsi
> @@ -287,7 +287,6 @@
> <0x49032000 0x7f>; /* L3 Interconnect */
> reg-names = "mpu", "dma";
> interrupts = <0 112 0x4>;
> - interrupt-parent = <&gic>;
> ti,hwmods = "mcpdm";
> };
>
> @@ -297,7 +296,6 @@
> <0x4902e000 0x7f>; /* L3 Interconnect */
> reg-names = "mpu", "dma";
> interrupts = <0 114 0x4>;
> - interrupt-parent = <&gic>;
> ti,hwmods = "dmic";
> };
>
> @@ -308,7 +306,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 17 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp1";
> };
> @@ -320,7 +317,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 22 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp2";
> };
> @@ -332,7 +328,6 @@
> reg-names = "mpu", "dma";
> interrupts = <0 23 0x4>;
> interrupt-names = "common";
> - interrupt-parent = <&gic>;
> ti,buffer-size = <128>;
> ti,hwmods = "mcbsp3";
> };
> --
> 1.7.0.4
>
More information about the devicetree-discuss
mailing list