| Message ID | 1496689941-20193-1-git-send-email-sunil.m@techveda.org |
|---|---|
| State | Changes Requested |
| Delegated to: | Tom Rini |
| Headers | show |
On Tue, Jun 06, 2017 at 12:42:21AM +0530, sunil.m@techveda.org wrote: > From: Suniel Mahesh <sunil.m@techveda.org> > > Interrupt-parent property is defined in the root node as > "interrupt-parent = <&intc>". This interrupt-parent value becomes > the default for the system, so removed redundant "interrupt-parent" > property from mmc, mac, lcdc and tscadc nodes. > > Signed-off-by: Suniel Mahesh <sunil.m@techveda.org> > --- > arch/arm/dts/am33xx.dtsi | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/arch/arm/dts/am33xx.dtsi b/arch/arm/dts/am33xx.dtsi > index b26e21b..14caee7 100644 > --- a/arch/arm/dts/am33xx.dtsi > +++ b/arch/arm/dts/am33xx.dtsi > @@ -315,7 +315,6 @@ > &edma 25>; > dma-names = "tx", "rx"; > interrupts = <64>; > - interrupt-parent = <&intc>; > reg = <0x48060000 0x1000>; > status = "disabled"; > }; > @@ -328,7 +327,6 @@ > &edma 3>; > dma-names = "tx", "rx"; > interrupts = <28>; > - interrupt-parent = <&intc>; > reg = <0x481d8000 0x1000>; > status = "disabled"; > }; > @@ -338,7 +336,6 @@ > ti,hwmods = "mmc3"; > ti,needs-special-reset; > interrupts = <29>; > - interrupt-parent = <&intc>; > reg = <0x47810000 0x1000>; > status = "disabled"; > }; > @@ -724,7 +721,6 @@ > 0x4a101200 0x100>; > #address-cells = <1>; > #size-cells = <1>; > - interrupt-parent = <&intc>; > /* > * c0_rx_thresh_pend > * c0_rx_pend > @@ -787,7 +783,6 @@ > lcdc: lcdc@4830e000 { > compatible = "ti,am33xx-tilcdc"; > reg = <0x4830e000 0x1000>; > - interrupt-parent = <&intc>; > interrupts = <36>; > ti,hwmods = "lcdc"; > status = "disabled"; > @@ -796,7 +791,6 @@ > tscadc: tscadc@44e0d000 { > compatible = "ti,am3359-tscadc"; > reg = <0x44e0d000 0x1000>; > - interrupt-parent = <&intc>; > interrupts = <16>; > ti,hwmods = "adc_tsc"; > status = "disabled"; Please submit this vs the kernel and we'll sync it in that way, thanks!
On Tuesday 06 June 2017 05:37 AM, Tom Rini wrote: > On Tue, Jun 06, 2017 at 12:42:21AM +0530, sunil.m@techveda.org wrote: > >> From: Suniel Mahesh <sunil.m@techveda.org> >> >> Interrupt-parent property is defined in the root node as >> "interrupt-parent = <&intc>". This interrupt-parent value becomes >> the default for the system, so removed redundant "interrupt-parent" >> property from mmc, mac, lcdc and tscadc nodes. >> >> Signed-off-by: Suniel Mahesh <sunil.m@techveda.org> >> --- >> arch/arm/dts/am33xx.dtsi | 6 ------ >> 1 file changed, 6 deletions(-) >> >> diff --git a/arch/arm/dts/am33xx.dtsi b/arch/arm/dts/am33xx.dtsi >> index b26e21b..14caee7 100644 >> --- a/arch/arm/dts/am33xx.dtsi >> +++ b/arch/arm/dts/am33xx.dtsi >> @@ -315,7 +315,6 @@ >> &edma 25>; >> dma-names = "tx", "rx"; >> interrupts = <64>; >> - interrupt-parent = <&intc>; >> reg = <0x48060000 0x1000>; >> status = "disabled"; >> }; >> @@ -328,7 +327,6 @@ >> &edma 3>; >> dma-names = "tx", "rx"; >> interrupts = <28>; >> - interrupt-parent = <&intc>; >> reg = <0x481d8000 0x1000>; >> status = "disabled"; >> }; >> @@ -338,7 +336,6 @@ >> ti,hwmods = "mmc3"; >> ti,needs-special-reset; >> interrupts = <29>; >> - interrupt-parent = <&intc>; >> reg = <0x47810000 0x1000>; >> status = "disabled"; >> }; >> @@ -724,7 +721,6 @@ >> 0x4a101200 0x100>; >> #address-cells = <1>; >> #size-cells = <1>; >> - interrupt-parent = <&intc>; >> /* >> * c0_rx_thresh_pend >> * c0_rx_pend >> @@ -787,7 +783,6 @@ >> lcdc: lcdc@4830e000 { >> compatible = "ti,am33xx-tilcdc"; >> reg = <0x4830e000 0x1000>; >> - interrupt-parent = <&intc>; >> interrupts = <36>; >> ti,hwmods = "lcdc"; >> status = "disabled"; >> @@ -796,7 +791,6 @@ >> tscadc: tscadc@44e0d000 { >> compatible = "ti,am3359-tscadc"; >> reg = <0x44e0d000 0x1000>; >> - interrupt-parent = <&intc>; >> interrupts = <16>; >> ti,hwmods = "adc_tsc"; >> status = "disabled"; > > Please submit this vs the kernel and we'll sync it in that way, thanks! > ok thanks for the suggestion, working on it.
diff --git a/arch/arm/dts/am33xx.dtsi b/arch/arm/dts/am33xx.dtsi index b26e21b..14caee7 100644 --- a/arch/arm/dts/am33xx.dtsi +++ b/arch/arm/dts/am33xx.dtsi @@ -315,7 +315,6 @@ &edma 25>; dma-names = "tx", "rx"; interrupts = <64>; - interrupt-parent = <&intc>; reg = <0x48060000 0x1000>; status = "disabled"; }; @@ -328,7 +327,6 @@ &edma 3>; dma-names = "tx", "rx"; interrupts = <28>; - interrupt-parent = <&intc>; reg = <0x481d8000 0x1000>; status = "disabled"; }; @@ -338,7 +336,6 @@ ti,hwmods = "mmc3"; ti,needs-special-reset; interrupts = <29>; - interrupt-parent = <&intc>; reg = <0x47810000 0x1000>; status = "disabled"; }; @@ -724,7 +721,6 @@ 0x4a101200 0x100>; #address-cells = <1>; #size-cells = <1>; - interrupt-parent = <&intc>; /* * c0_rx_thresh_pend * c0_rx_pend @@ -787,7 +783,6 @@ lcdc: lcdc@4830e000 { compatible = "ti,am33xx-tilcdc"; reg = <0x4830e000 0x1000>; - interrupt-parent = <&intc>; interrupts = <36>; ti,hwmods = "lcdc"; status = "disabled"; @@ -796,7 +791,6 @@ tscadc: tscadc@44e0d000 { compatible = "ti,am3359-tscadc"; reg = <0x44e0d000 0x1000>; - interrupt-parent = <&intc>; interrupts = <16>; ti,hwmods = "adc_tsc"; status = "disabled";