U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property
@ 2017-08-10 10:10 sunil.m at techveda.org
  2017-08-10 23:01 ` Tom Rini
  0 siblings, 1 reply; 5+ messages in thread
From: sunil.m at techveda.org @ 2017-08-10 10:10 UTC (permalink / raw)
  To: u-boot

From: Suniel Mahesh <sunil.m@techveda.org>

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 <sunil.m@techveda.org>
---
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 at 4830e000 {
 			compatible = "ti,am33xx-tilcdc";
 			reg = <0x4830e000 0x1000>;
-			interrupt-parent = <&intc>;
 			interrupts = <36>;
 			ti,hwmods = "lcdc";
 			status = "disabled";
@@ -796,7 +791,6 @@
 		tscadc: tscadc at 44e0d000 {
 			compatible = "ti,am3359-tscadc";
 			reg = <0x44e0d000 0x1000>;
-			interrupt-parent = <&intc>;
 			interrupts = <16>;
 			ti,hwmods = "adc_tsc";
 			status = "disabled";
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property
  2017-08-10 10:10 [U-Boot] [RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property sunil.m at techveda.org
@ 2017-08-10 23:01 ` Tom Rini
  2017-08-12  7:56   ` [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4 sunil.m at techveda.org
  0 siblings, 1 reply; 5+ messages in thread
From: Tom Rini @ 2017-08-10 23:01 UTC (permalink / raw)
  To: u-boot

On Thu, Aug 10, 2017 at 03:40:24PM +0530, sunil.m at techveda.org wrote:

> From: Suniel Mahesh <sunil.m@techveda.org>
> 
> 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 <sunil.m@techveda.org>
> ---
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170810/14ba18bb/attachment.sig>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4
  2017-08-10 23:01 ` Tom Rini
@ 2017-08-12  7:56   ` sunil.m at techveda.org
  2017-08-13 21:20     ` Tom Rini
  2017-08-26 20:45     ` [U-Boot] [U-Boot, " Tom Rini
  0 siblings, 2 replies; 5+ messages in thread
From: sunil.m at techveda.org @ 2017-08-12  7:56 UTC (permalink / raw)
  To: u-boot

From: Suniel Mahesh <sunil.m@techveda.org>

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 <sunil.m@techveda.org>
---
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 at 4830e000 {
 			compatible = "ti,am33xx-tilcdc";
 			reg = <0x4830e000 0x1000>;
-			interrupt-parent = <&intc>;
 			interrupts = <36>;
 			ti,hwmods = "lcdc";
 			status = "disabled";
@@ -796,7 +791,6 @@
 		tscadc: tscadc at 44e0d000 {
 			compatible = "ti,am3359-tscadc";
 			reg = <0x44e0d000 0x1000>;
-			interrupt-parent = <&intc>;
 			interrupts = <16>;
 			ti,hwmods = "adc_tsc";
 			status = "disabled";
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4
  2017-08-12  7:56   ` [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4 sunil.m at techveda.org
@ 2017-08-13 21:20     ` Tom Rini
  2017-08-26 20:45     ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2017-08-13 21:20 UTC (permalink / raw)
  To: u-boot

On Sat, Aug 12, 2017 at 01:26:38PM +0530, sunil.m at techveda.org wrote:

> From: Suniel Mahesh <sunil.m@techveda.org>
> 
> 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 <sunil.m@techveda.org>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170813/f7b91947/attachment.sig>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [U-Boot, v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4
  2017-08-12  7:56   ` [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4 sunil.m at techveda.org
  2017-08-13 21:20     ` Tom Rini
@ 2017-08-26 20:45     ` Tom Rini
  1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2017-08-26 20:45 UTC (permalink / raw)
  To: u-boot

On Sat, Aug 12, 2017 at 01:26:38PM +0530, sunil.m at techveda.org wrote:

> From: Suniel Mahesh <sunil.m@techveda.org>
> 
> 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 <sunil.m@techveda.org>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170826/13ae55ad/attachment.sig>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-08-26 20:45 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-10 10:10 [U-Boot] [RESEND PATCH] arm: dts: am33xx: Remove redundant interrupt-parent property sunil.m at techveda.org
2017-08-10 23:01 ` Tom Rini
2017-08-12  7:56   ` [U-Boot] [PATCH v2] arm: dts: am33xx: sync DTS with Linux 4.13-rc4 sunil.m at techveda.org
2017-08-13 21:20     ` Tom Rini
2017-08-26 20:45     ` [U-Boot] [U-Boot, " Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox