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@xxxxxx> > Cc: Vaibhav Hiremath <hvaibhav@xxxxxx> > Cc: Peter Ujfalusi <peter.ujfalusi@xxxxxx> > Cc: Sebastien Guiriec <s-guiriec@xxxxxx> > --- > > 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@xxxxxx> 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 > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
