diff mbox series

arm64: tegra: Disable XUSB support on Jetson TX2

Message ID 20190506164121.4507-1-thierry.reding@gmail.com
State Deferred
Headers show
Series arm64: tegra: Disable XUSB support on Jetson TX2 | expand

Commit Message

Thierry Reding May 6, 2019, 4:41 p.m. UTC
From: Thierry Reding <treding@nvidia.com>

The recently introduced XUSB support for Jetson TX2 is causing boot, CPU
hotplug and suspend/resume failures according to several reports.

Temporarily work around this by disabling the XUSB controller and XUSB
pad controller nodes in device tree, while we figure out what's causing
this.

Reported-by: Bitan Biswas <bbiswas@nvidia.com>
Reported-by: Jonathan Hunter <jonathanh@nvidia.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
 arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Bitan Biswas May 7, 2019, 4:19 a.m. UTC | #1
On 5/6/19 9:41 AM, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> The recently introduced XUSB support for Jetson TX2 is causing boot, CPU
> hotplug and suspend/resume failures according to several reports.
> 
> Temporarily work around this by disabling the XUSB controller and XUSB
> pad controller nodes in device tree, while we figure out what's causing
> this.
> 
> Reported-by: Bitan Biswas <bbiswas@nvidia.com>
> Reported-by: Jonathan Hunter <jonathanh@nvidia.com>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
>   arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> index 75ee6cf1e1b4..14d7fea82daf 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> @@ -59,7 +59,7 @@
>   	};
>   
>   	padctl@3520000 {
> -		status = "okay";
> +		status = "disabled";
>   
>   		avdd-pll-erefeut-supply = <&vdd_1v8_pll>;
>   		avdd-usb-supply = <&vdd_3v3_sys>;
> @@ -137,7 +137,7 @@
>   	};
>   
>   	usb@3530000 {
> -		status = "okay";
> +		status = "disabled";
>   
>   		phys = <&{/padctl@3520000/pads/usb2/lanes/usb2-0}>,
>   		       <&{/padctl@3520000/pads/usb2/lanes/usb2-1}>,
> 
Verified with this patch that Jetson TX2 core power is again turning off 
and suspend mode is entered with additional Tegra186 suspend patches 
from below patch list under review :
http://patchwork.ozlabs.org/project/linux-tegra/list/?series=78866
Jon Hunter May 8, 2019, 10:53 a.m. UTC | #2
On 06/05/2019 17:41, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> The recently introduced XUSB support for Jetson TX2 is causing boot, CPU
> hotplug and suspend/resume failures according to several reports.
> 
> Temporarily work around this by disabling the XUSB controller and XUSB
> pad controller nodes in device tree, while we figure out what's causing
> this.
> 
> Reported-by: Bitan Biswas <bbiswas@nvidia.com>
> Reported-by: Jonathan Hunter <jonathanh@nvidia.com>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
>  arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> index 75ee6cf1e1b4..14d7fea82daf 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
> @@ -59,7 +59,7 @@
>  	};
>  
>  	padctl@3520000 {
> -		status = "okay";
> +		status = "disabled";
>  
>  		avdd-pll-erefeut-supply = <&vdd_1v8_pll>;
>  		avdd-usb-supply = <&vdd_3v3_sys>;
> @@ -137,7 +137,7 @@
>  	};
>  
>  	usb@3530000 {
> -		status = "okay";
> +		status = "disabled";
>  
>  		phys = <&{/padctl@3520000/pads/usb2/lanes/usb2-0}>,
>  		       <&{/padctl@3520000/pads/usb2/lanes/usb2-1}>,
> 

Thanks.

Tested-by: Jon Hunter <jonathanh@nvidia.com>
Acked-by: Jon Hunter <jonathanh@nvidia.com>

Cheers
Jon
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
index 75ee6cf1e1b4..14d7fea82daf 100644
--- a/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
+++ b/arch/arm64/boot/dts/nvidia/tegra186-p2771-0000.dts
@@ -59,7 +59,7 @@ 
 	};
 
 	padctl@3520000 {
-		status = "okay";
+		status = "disabled";
 
 		avdd-pll-erefeut-supply = <&vdd_1v8_pll>;
 		avdd-usb-supply = <&vdd_3v3_sys>;
@@ -137,7 +137,7 @@ 
 	};
 
 	usb@3530000 {
-		status = "okay";
+		status = "disabled";
 
 		phys = <&{/padctl@3520000/pads/usb2/lanes/usb2-0}>,
 		       <&{/padctl@3520000/pads/usb2/lanes/usb2-1}>,