[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