linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] ARM: dts: dra7: Fix duplicate USB4 device node
@ 2021-05-26 17:20 Gowtham Tammana
  2021-05-26 18:01 ` Suman Anna
  0 siblings, 1 reply; 3+ messages in thread
From: Gowtham Tammana @ 2021-05-26 17:20 UTC (permalink / raw)
  To: tony, bcousson, Suman Anna
  Cc: robh+dt, linux-omap, devicetree, linux-kernel, linux-arm-kernel,
	Grygorii Strashko, Nisanth Menon, Gowtham Tammana

With [1] USB4 device node got defined in dra74x.dtsi file. However,
there was a prior defintion of the same in [2] which didn't get removed
causing boot failures. USB4 node is present only in DRA74x variants so
keeping the entry in dra74x.dtsi and removing it from the top level
interconnect hierarchy dra7-l4.dtsi file.

Since USB4 is only included in DRA74x variants, remove its reference
for AM5718, DRA71x and DR72x boards.

[1]: commit 549fce068a311 ("ARM: dts: dra7: Add l4 interconnect
hierarchy and ti-sysc data")
[2]: commit c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for
dra7 dwc3")

Fixes: c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for dra7 dwc3")
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
v2:
  - changed reference to commit sha instead of line numbers
  - added Fixes: tag
  - moved the definition to dra74.dtsi as per Suman and Tony review comments

 arch/arm/boot/dts/am5718.dtsi  |  6 +-----
 arch/arm/boot/dts/dra7-l4.dtsi | 22 ----------------------
 arch/arm/boot/dts/dra71x.dtsi  |  4 ----
 arch/arm/boot/dts/dra72x.dtsi  |  4 ----
 arch/arm/boot/dts/dra74x.dtsi  |  2 +-
 5 files changed, 2 insertions(+), 36 deletions(-)

diff --git a/arch/arm/boot/dts/am5718.dtsi b/arch/arm/boot/dts/am5718.dtsi
index ebf4d3cc1cfb..6d7530a48c73 100644
--- a/arch/arm/boot/dts/am5718.dtsi
+++ b/arch/arm/boot/dts/am5718.dtsi
@@ -17,17 +17,13 @@ / {
  * VCP1, VCP2
  * MLB
  * ISS
- * USB3, USB4
+ * USB3
  */
 
 &usb3_tm {
 	status = "disabled";
 };
 
-&usb4_tm {
-	status = "disabled";
-};
-
 &atl_tm {
 	status = "disabled";
 };
diff --git a/arch/arm/boot/dts/dra7-l4.dtsi b/arch/arm/boot/dts/dra7-l4.dtsi
index 149144cdff35..648d23f7f748 100644
--- a/arch/arm/boot/dts/dra7-l4.dtsi
+++ b/arch/arm/boot/dts/dra7-l4.dtsi
@@ -4129,28 +4129,6 @@ usb3: usb@10000 {
 			};
 		};
 
-		usb4_tm: target-module@140000 {		/* 0x48940000, ap 75 3c.0 */
-			compatible = "ti,sysc-omap4", "ti,sysc";
-			reg = <0x140000 0x4>,
-			      <0x140010 0x4>;
-			reg-names = "rev", "sysc";
-			ti,sysc-mask = <SYSC_OMAP4_DMADISABLE>;
-			ti,sysc-midle = <SYSC_IDLE_FORCE>,
-					<SYSC_IDLE_NO>,
-					<SYSC_IDLE_SMART>,
-					<SYSC_IDLE_SMART_WKUP>;
-			ti,sysc-sidle = <SYSC_IDLE_FORCE>,
-					<SYSC_IDLE_NO>,
-					<SYSC_IDLE_SMART>,
-					<SYSC_IDLE_SMART_WKUP>;
-			/* Domains (P, C): l3init_pwrdm, l3init_clkdm */
-			clocks = <&l3init_clkctrl DRA7_L3INIT_USB_OTG_SS4_CLKCTRL 0>;
-			clock-names = "fck";
-			#address-cells = <1>;
-			#size-cells = <1>;
-			ranges = <0x0 0x140000 0x20000>;
-		};
-
 		target-module@170000 {			/* 0x48970000, ap 21 0a.0 */
 			compatible = "ti,sysc-omap4", "ti,sysc";
 			reg = <0x170010 0x4>;
diff --git a/arch/arm/boot/dts/dra71x.dtsi b/arch/arm/boot/dts/dra71x.dtsi
index cad0e4a2bd8d..9c270d8f75d5 100644
--- a/arch/arm/boot/dts/dra71x.dtsi
+++ b/arch/arm/boot/dts/dra71x.dtsi
@@ -11,7 +11,3 @@
 &rtctarget {
 	status = "disabled";
 };
-
-&usb4_tm {
-	status = "disabled";
-};
diff --git a/arch/arm/boot/dts/dra72x.dtsi b/arch/arm/boot/dts/dra72x.dtsi
index d403acc754b6..f3e934ef7d3e 100644
--- a/arch/arm/boot/dts/dra72x.dtsi
+++ b/arch/arm/boot/dts/dra72x.dtsi
@@ -108,7 +108,3 @@ &pcie1_ep {
 &pcie2_rc {
 	compatible = "ti,dra726-pcie-rc", "ti,dra7-pcie";
 };
-
-&usb4_tm {
-	status = "disabled";
-};
diff --git a/arch/arm/boot/dts/dra74x.dtsi b/arch/arm/boot/dts/dra74x.dtsi
index e1850d6c841a..60f2ab8d34d5 100644
--- a/arch/arm/boot/dts/dra74x.dtsi
+++ b/arch/arm/boot/dts/dra74x.dtsi
@@ -49,7 +49,7 @@ dsp2_system: dsp_system@41500000 {
 			reg = <0x41500000 0x100>;
 		};
 
-		target-module@48940000 {
+		usb4_tm: target-module@48940000 {
 			compatible = "ti,sysc-omap4", "ti,sysc";
 			reg = <0x48940000 0x4>,
 			      <0x48940010 0x4>;
-- 
2.31.1


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

* Re: [PATCH v2] ARM: dts: dra7: Fix duplicate USB4 device node
  2021-05-26 17:20 [PATCH v2] ARM: dts: dra7: Fix duplicate USB4 device node Gowtham Tammana
@ 2021-05-26 18:01 ` Suman Anna
  2021-05-26 19:48   ` Tammana, Gowtham
  0 siblings, 1 reply; 3+ messages in thread
From: Suman Anna @ 2021-05-26 18:01 UTC (permalink / raw)
  To: Gowtham Tammana, tony, bcousson
  Cc: robh+dt, linux-omap, devicetree, linux-kernel, linux-arm-kernel,
	Grygorii Strashko, Nisanth Menon

Hi Gowtham,

On 5/26/21 12:20 PM, Gowtham Tammana wrote:
> With [1] USB4 device node got defined in dra74x.dtsi file. However,
> there was a prior defintion of the same in [2] which didn't get removed

I like this approach and is more in line with the original move of USB4 node
into dra74x.dtsi file.

Looks like you mixed up [1] and [2] or I misunderstand your terminology. Also,
let's be precise, it is the USB4 target-module node that got duplicated in [2],
USB4 node always existed but outside of ti-sysc control before [2] and did not
conflict with the target-module node added in [1].

> causing boot failures. 

on what boards? Is it affecting all of them?

regards
Suman

USB4 node is present only in DRA74x variants so
> keeping the entry in dra74x.dtsi and removing it from the top level
> interconnect hierarchy dra7-l4.dtsi file.
> 
> Since USB4 is only included in DRA74x variants, remove its reference
> for AM5718, DRA71x and DR72x boards.
> 
> [1]: commit 549fce068a311 ("ARM: dts: dra7: Add l4 interconnect
> hierarchy and ti-sysc data")
> [2]: commit c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for
> dra7 dwc3")
> 
> Fixes: c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for dra7 dwc3")
> Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
> ---
> v2:
>   - changed reference to commit sha instead of line numbers
>   - added Fixes: tag
>   - moved the definition to dra74.dtsi as per Suman and Tony review comments
> 
>  arch/arm/boot/dts/am5718.dtsi  |  6 +-----
>  arch/arm/boot/dts/dra7-l4.dtsi | 22 ----------------------
>  arch/arm/boot/dts/dra71x.dtsi  |  4 ----
>  arch/arm/boot/dts/dra72x.dtsi  |  4 ----
>  arch/arm/boot/dts/dra74x.dtsi  |  2 +-
>  5 files changed, 2 insertions(+), 36 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/am5718.dtsi b/arch/arm/boot/dts/am5718.dtsi
> index ebf4d3cc1cfb..6d7530a48c73 100644
> --- a/arch/arm/boot/dts/am5718.dtsi
> +++ b/arch/arm/boot/dts/am5718.dtsi
> @@ -17,17 +17,13 @@ / {
>   * VCP1, VCP2
>   * MLB
>   * ISS
> - * USB3, USB4
> + * USB3
>   */
>  
>  &usb3_tm {
>  	status = "disabled";
>  };
>  
> -&usb4_tm {
> -	status = "disabled";
> -};
> -
>  &atl_tm {
>  	status = "disabled";
>  };
> diff --git a/arch/arm/boot/dts/dra7-l4.dtsi b/arch/arm/boot/dts/dra7-l4.dtsi
> index 149144cdff35..648d23f7f748 100644
> --- a/arch/arm/boot/dts/dra7-l4.dtsi
> +++ b/arch/arm/boot/dts/dra7-l4.dtsi
> @@ -4129,28 +4129,6 @@ usb3: usb@10000 {
>  			};
>  		};
>  
> -		usb4_tm: target-module@140000 {		/* 0x48940000, ap 75 3c.0 */
> -			compatible = "ti,sysc-omap4", "ti,sysc";
> -			reg = <0x140000 0x4>,
> -			      <0x140010 0x4>;
> -			reg-names = "rev", "sysc";
> -			ti,sysc-mask = <SYSC_OMAP4_DMADISABLE>;
> -			ti,sysc-midle = <SYSC_IDLE_FORCE>,
> -					<SYSC_IDLE_NO>,
> -					<SYSC_IDLE_SMART>,
> -					<SYSC_IDLE_SMART_WKUP>;
> -			ti,sysc-sidle = <SYSC_IDLE_FORCE>,
> -					<SYSC_IDLE_NO>,
> -					<SYSC_IDLE_SMART>,
> -					<SYSC_IDLE_SMART_WKUP>;
> -			/* Domains (P, C): l3init_pwrdm, l3init_clkdm */
> -			clocks = <&l3init_clkctrl DRA7_L3INIT_USB_OTG_SS4_CLKCTRL 0>;
> -			clock-names = "fck";
> -			#address-cells = <1>;
> -			#size-cells = <1>;
> -			ranges = <0x0 0x140000 0x20000>;
> -		};
> -
>  		target-module@170000 {			/* 0x48970000, ap 21 0a.0 */
>  			compatible = "ti,sysc-omap4", "ti,sysc";
>  			reg = <0x170010 0x4>;
> diff --git a/arch/arm/boot/dts/dra71x.dtsi b/arch/arm/boot/dts/dra71x.dtsi
> index cad0e4a2bd8d..9c270d8f75d5 100644
> --- a/arch/arm/boot/dts/dra71x.dtsi
> +++ b/arch/arm/boot/dts/dra71x.dtsi
> @@ -11,7 +11,3 @@
>  &rtctarget {
>  	status = "disabled";
>  };
> -
> -&usb4_tm {
> -	status = "disabled";
> -};
> diff --git a/arch/arm/boot/dts/dra72x.dtsi b/arch/arm/boot/dts/dra72x.dtsi
> index d403acc754b6..f3e934ef7d3e 100644
> --- a/arch/arm/boot/dts/dra72x.dtsi
> +++ b/arch/arm/boot/dts/dra72x.dtsi
> @@ -108,7 +108,3 @@ &pcie1_ep {
>  &pcie2_rc {
>  	compatible = "ti,dra726-pcie-rc", "ti,dra7-pcie";
>  };
> -
> -&usb4_tm {
> -	status = "disabled";
> -};
> diff --git a/arch/arm/boot/dts/dra74x.dtsi b/arch/arm/boot/dts/dra74x.dtsi
> index e1850d6c841a..60f2ab8d34d5 100644
> --- a/arch/arm/boot/dts/dra74x.dtsi
> +++ b/arch/arm/boot/dts/dra74x.dtsi
> @@ -49,7 +49,7 @@ dsp2_system: dsp_system@41500000 {
>  			reg = <0x41500000 0x100>;
>  		};
>  
> -		target-module@48940000 {
> +		usb4_tm: target-module@48940000 {
>  			compatible = "ti,sysc-omap4", "ti,sysc";
>  			reg = <0x48940000 0x4>,
>  			      <0x48940010 0x4>;
> 


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

* RE: [PATCH v2] ARM: dts: dra7: Fix duplicate USB4 device node
  2021-05-26 18:01 ` Suman Anna
@ 2021-05-26 19:48   ` Tammana, Gowtham
  0 siblings, 0 replies; 3+ messages in thread
From: Tammana, Gowtham @ 2021-05-26 19:48 UTC (permalink / raw)
  To: Anna, Suman, tony, bcousson
  Cc: robh+dt, linux-omap, devicetree, linux-kernel, linux-arm-kernel,
	Strashko, Grygorii, Menon, Nishanth



>-----Original Message-----
>From: Anna, Suman
>Sent: Wednesday, May 26, 2021 1:01 PM
>
>Hi Gowtham,
>
>On 5/26/21 12:20 PM, Gowtham Tammana wrote:
>> With [1] USB4 device node got defined in dra74x.dtsi file. However,
>> there was a prior defintion of the same in [2] which didn't get removed
>
>I like this approach and is more in line with the original move of USB4
>node
>into dra74x.dtsi file.
>
>Looks like you mixed up [1] and [2] or I misunderstand your terminology.

Yes, 1 and 2 got mixed up here. Will post a v3.

>Also,
>let's be precise, it is the USB4 target-module node that got duplicated in
>[2],
>USB4 node always existed but outside of ti-sysc control before [2] and did
>not
>conflict with the target-module node added in [1].
>
>> causing boot failures.
>
>on what boards? Is it affecting all of them?

I have a dra7-evm board and see failure on it. This should also affect
Dra76-evm, am572x beagle boards, am574x as they all pull in dra74x.dtsi.
Will mention these in the v3 update.

Thanks,
Gowtham 

> USB4 node is present only in DRA74x variants so
>> keeping the entry in dra74x.dtsi and removing it from the top level
>> interconnect hierarchy dra7-l4.dtsi file.
>>
>> Since USB4 is only included in DRA74x variants, remove its reference
>> for AM5718, DRA71x and DR72x boards.
>>
>> [1]: commit 549fce068a311 ("ARM: dts: dra7: Add l4 interconnect
>> hierarchy and ti-sysc data")
>> [2]: commit c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for
>> dra7 dwc3")
>>
>> Fixes: c7b72abca61ec ("ARM: OMAP2+: Drop legacy platform data for dra7
>dwc3")
>> Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
>> ---
>> v2:

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

end of thread, other threads:[~2021-05-26 19:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-26 17:20 [PATCH v2] ARM: dts: dra7: Fix duplicate USB4 device node Gowtham Tammana
2021-05-26 18:01 ` Suman Anna
2021-05-26 19:48   ` Tammana, Gowtham

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).