Re: [PATCH] arm64: tegra: Enable Jetson-Xavier J512 USB host

2021-01-21 Thread Thierry Reding 
On Tue, Jan 19, 2021 at 10:23:49AM +0800, JC Kuo wrote:
> This commit enables USB host mode at J512 type-C port of Jetson-Xavier.
> 
> Signed-off-by: JC Kuo 
> ---
>  .../arm64/boot/dts/nvidia/tegra194-p2888.dtsi |  8 +++
>  .../boot/dts/nvidia/tegra194-p2972-.dts   | 24 +--
>  2 files changed, 30 insertions(+), 2 deletions(-)

Applied, thanks.

Thierry


signature.asc
Description: PGP signature

Re: [PATCH] arm64: tegra: Enable Jetson-Xavier J512 USB host

2021-01-21 Thread Jon Hunter 


On 19/01/2021 02:23, JC Kuo wrote:
> This commit enables USB host mode at J512 type-C port of Jetson-Xavier.
> 
> Signed-off-by: JC Kuo 
> ---
>  .../arm64/boot/dts/nvidia/tegra194-p2888.dtsi |  8 +++
>  .../boot/dts/nvidia/tegra194-p2972-.dts   | 24 +--
>  2 files changed, 30 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi 
> b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> index d71b7a1140fe..7e7b0eb90c80 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> @@ -93,6 +93,10 @@ padctl@352 {
>   vclamp-usb-supply = <_1v8ao>;
>  
>   ports {
> + usb2-0 {
> + vbus-supply = <_5v0_sys>;
> + };
> +
>   usb2-1 {
>   vbus-supply = <_5v0_sys>;
>   };
> @@ -105,6 +109,10 @@ usb3-0 {
>   vbus-supply = <_5v0_sys>;
>   };
>  
> + usb3-2 {
> + vbus-supply = <_5v0_sys>;
> + };
> +
>   usb3-3 {
>   vbus-supply = <_5v0_sys>;
>   };
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts 
> b/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
> index 54d057beec59..8697927b1fe7 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
> @@ -57,6 +57,10 @@ padctl@352 {
>   pads {
>   usb2 {
>   lanes {
> + usb2-0 {
> + status = "okay";
> + };
> +
>   usb2-1 {
>   status = "okay";
>   };
> @@ -73,6 +77,10 @@ usb3-0 {
>   status = "okay";
>   };
>  
> + usb3-2 {
> + status = "okay";
> + };
> +
>   usb3-3 {
>   status = "okay";
>   };
> @@ -81,6 +89,11 @@ usb3-3 {
>   };
>  
>   ports {
> + usb2-0 {
> + mode = "host";
> + status = "okay";
> + };
> +
>   usb2-1 {
>   mode = "host";
>   status = "okay";
> @@ -96,6 +109,11 @@ usb3-0 {
>   status = "okay";
>   };
>  
> + usb3-2 {
> + nvidia,usb2-companion = <0>;
> + status = "okay";
> + };
> +
>   usb3-3 {
>   nvidia,usb2-companion = <3>;
>   maximum-speed = "super-speed";
> @@ -107,11 +125,13 @@ usb3-3 {
>   usb@361 {
>   status = "okay";
>  
> - phys =  
> <&{/bus@0/padctl@352/pads/usb2/lanes/usb2-1}>,
> + phys =  
> <&{/bus@0/padctl@352/pads/usb2/lanes/usb2-0}>,
> + 
> <&{/bus@0/padctl@352/pads/usb2/lanes/usb2-1}>,
>   
> <&{/bus@0/padctl@352/pads/usb2/lanes/usb2-3}>,
>   
> <&{/bus@0/padctl@352/pads/usb3/lanes/usb3-0}>,
> + 
> <&{/bus@0/padctl@352/pads/usb3/lanes/usb3-2}>,
>   
> <&{/bus@0/padctl@352/pads/usb3/lanes/usb3-3}>;
> - phy-names = "usb2-1", "usb2-3", "usb3-0", "usb3-3";
> + phy-names = "usb2-0", "usb2-1", "usb2-3", "usb3-0", 
> "usb3-2", "usb3-3";
>   };
>  
>   pwm@c34 {
> 

Thanks. Works for me.

Acked-by: Jon Hunter 
Tested-by: Jon Hunter 

Cheers
Jon

-- 
nvpublic

[PATCH] arm64: tegra: Enable Jetson-Xavier J512 USB host

2021-01-18 Thread JC Kuo 
This commit enables USB host mode at J512 type-C port of Jetson-Xavier.

Signed-off-by: JC Kuo 
---
 .../arm64/boot/dts/nvidia/tegra194-p2888.dtsi |  8 +++
 .../boot/dts/nvidia/tegra194-p2972-.dts   | 24 +--
 2 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi 
b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
index d71b7a1140fe..7e7b0eb90c80 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
@@ -93,6 +93,10 @@ padctl@352 {
vclamp-usb-supply = <_1v8ao>;
 
ports {
+   usb2-0 {
+   vbus-supply = <_5v0_sys>;
+   };
+
usb2-1 {
vbus-supply = <_5v0_sys>;
};
@@ -105,6 +109,10 @@ usb3-0 {
vbus-supply = <_5v0_sys>;
};
 
+   usb3-2 {
+   vbus-supply = <_5v0_sys>;
+   };
+
usb3-3 {
vbus-supply = <_5v0_sys>;
};
diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts 
b/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
index 54d057beec59..8697927b1fe7 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
+++ b/arch/arm64/boot/dts/nvidia/tegra194-p2972-.dts
@@ -57,6 +57,10 @@ padctl@352 {
pads {
usb2 {
lanes {
+   usb2-0 {
+   status = "okay";
+   };
+
usb2-1 {
status = "okay";
};
@@ -73,6 +77,10 @@ usb3-0 {
status = "okay";
};
 
+   usb3-2 {
+   status = "okay";
+   };
+
usb3-3 {
status = "okay";
};
@@ -81,6 +89,11 @@ usb3-3 {
};
 
ports {
+   usb2-0 {
+   mode = "host";
+   status = "okay";
+   };
+
usb2-1 {
mode = "host";
status = "okay";
@@ -96,6 +109,11 @@ usb3-0 {
status = "okay";
};
 
+   usb3-2 {
+   nvidia,usb2-companion = <0>;
+   status = "okay";
+   };
+
usb3-3 {
nvidia,usb2-companion = <3>;
maximum-speed = "super-speed";
@@ -107,11 +125,13 @@ usb3-3 {
usb@361 {
status = "okay";
 
-   phys =  
<&{/bus@0/padctl@352/pads/usb2/lanes/usb2-1}>,
+   phys =  
<&{/bus@0/padctl@352/pads/usb2/lanes/usb2-0}>,
+   
<&{/bus@0/padctl@352/pads/usb2/lanes/usb2-1}>,

<&{/bus@0/padctl@352/pads/usb2/lanes/usb2-3}>,

<&{/bus@0/padctl@352/pads/usb3/lanes/usb3-0}>,
+   
<&{/bus@0/padctl@352/pads/usb3/lanes/usb3-2}>,

<&{/bus@0/padctl@352/pads/usb3/lanes/usb3-3}>;
-   phy-names = "usb2-1", "usb2-3", "usb3-0", "usb3-3";
+   phy-names = "usb2-0", "usb2-1", "usb2-3", "usb3-0", 
"usb3-2", "usb3-3";
};
 
pwm@c34 {
-- 
2.25.1