[RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property

Suniel Mahesh
From: Suniel Mahesh <[hidden email]>

Upstream Linux has the Interrupt-parent property being removed
from mmc, mac, lcdc and tscadc sub nodes in the dtsi file.
Interrupt-parent property is already defined in the root node.
Therefore, update the dtsi to mimic this change and remove duplicates.

Signed-off-by: Suniel Mahesh <[hidden email]>
---
Note:
- Compile tested on latest u-boot mainline tree no build issues.
- commit information upstream Linux:
  arm: dts: am33xx: Remove redundant interrupt-parent property
  sha1: de09eb52a1cceb6f80464a008c67c7bebb242314
---
 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";
--
1.9.1

_______________________________________________
U-Boot mailing list
[hidden email]
https://lists.denx.de/listinfo/u-boot
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property

Tom Rini-4
On Thu, Aug 10, 2017 at 03:40:24PM +0530, [hidden email] wrote:

> From: Suniel Mahesh <[hidden email]>
>
> Upstream Linux has the Interrupt-parent property being removed
> from mmc, mac, lcdc and tscadc sub nodes in the dtsi file.
> Interrupt-parent property is already defined in the root node.
> Therefore, update the dtsi to mimic this change and remove duplicates.
>
> Signed-off-by: Suniel Mahesh <[hidden email]>
> ---
> Note:
> - Compile tested on latest u-boot mainline tree no build issues.
> - commit information upstream Linux:
>   arm: dts: am33xx: Remove redundant interrupt-parent property
>   sha1: de09eb52a1cceb6f80464a008c67c7bebb242314
Can you please re-sync with that commit ID for all am33xx DTS files and
mention it (or perhaps v4.13-rc3 or so) in the commit message?  Thanks!

--
Tom

_______________________________________________
U-Boot mailing list
[hidden email]
https://lists.denx.de/listinfo/u-boot

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4

Suniel Mahesh
From: Suniel Mahesh <[hidden email]>

This re-syncs AM33xx DTS file with current file from
Linux v4.13-rc4 to ensure a consistent configuration. Upstream
Linux removed the redundant Interrupt-parent property from mmc,
mac, lcdc and tscadc sub nodes.

Signed-off-by: Suniel Mahesh <[hidden email]>
---
Changes for v2:
- Made corrections as suggested by Tom Rini
- Rebased and compile tested on latest u-boot mainline tree no build issues.

Note:
- commit information upstream Linux:
  arm: dts: am33xx: Remove redundant interrupt-parent property
  sha1: de09eb52a1cceb6f80464a008c67c7bebb242314
---
 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";
--
1.9.1

_______________________________________________
U-Boot mailing list
[hidden email]
https://lists.denx.de/listinfo/u-boot
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4

Tom Rini-4
On Sat, Aug 12, 2017 at 01:26:38PM +0530, [hidden email] wrote:

> From: Suniel Mahesh <[hidden email]>
>
> This re-syncs AM33xx DTS file with current file from
> Linux v4.13-rc4 to ensure a consistent configuration. Upstream
> Linux removed the redundant Interrupt-parent property from mmc,
> mac, lcdc and tscadc sub nodes.
>
> Signed-off-by: Suniel Mahesh <[hidden email]>

Reviewed-by: Tom Rini <[hidden email]>

--
Tom

_______________________________________________
U-Boot mailing list
[hidden email]
https://lists.denx.de/listinfo/u-boot

signature.asc (836 bytes) Download Attachment
Loading...