lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4790de5a-f299-219b-1d50-6fe98942c2e1@nvidia.com>
Date:   Thu, 21 Jan 2021 10:26:23 +0000
From:   Jon Hunter <jonathanh@...dia.com>
To:     JC Kuo <jckuo@...dia.com>, <gregkh@...uxfoundation.org>,
        <thierry.reding@...il.com>, <robh+dt@...nel.org>
CC:     <linux-kernel@...r.kernel.org>, <linux-tegra@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <stable@...r.kernel.org>
Subject: Re: [PATCH] arm64: tegra: Enable Jetson-Xavier J512 USB host


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 <jckuo@...dia.com>
> ---
>  .../arm64/boot/dts/nvidia/tegra194-p2888.dtsi |  8 +++++++
>  .../boot/dts/nvidia/tegra194-p2972-0000.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@...0000 {
>  			vclamp-usb-supply = <&vdd_1v8ao>;
>  
>  			ports {
> +				usb2-0 {
> +					vbus-supply = <&vdd_5v0_sys>;
> +				};
> +
>  				usb2-1 {
>  					vbus-supply = <&vdd_5v0_sys>;
>  				};
> @@ -105,6 +109,10 @@ usb3-0 {
>  					vbus-supply = <&vdd_5v0_sys>;
>  				};
>  
> +				usb3-2 {
> +					vbus-supply = <&vdd_5v0_sys>;
> +				};
> +
>  				usb3-3 {
>  					vbus-supply = <&vdd_5v0_sys>;
>  				};
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts b/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts
> index 54d057beec59..8697927b1fe7 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts
> @@ -57,6 +57,10 @@ padctl@...0000 {
>  			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@...0000 {
>  			status = "okay";
>  
> -			phys =	<&{/bus@...adctl@...0000/pads/usb2/lanes/usb2-1}>,
> +			phys =	<&{/bus@...adctl@...0000/pads/usb2/lanes/usb2-0}>,
> +				<&{/bus@...adctl@...0000/pads/usb2/lanes/usb2-1}>,
>  				<&{/bus@...adctl@...0000/pads/usb2/lanes/usb2-3}>,
>  				<&{/bus@...adctl@...0000/pads/usb3/lanes/usb3-0}>,
> +				<&{/bus@...adctl@...0000/pads/usb3/lanes/usb3-2}>,
>  				<&{/bus@...adctl@...0000/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@...0000 {
> 

Thanks. Works for me.

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

Cheers
Jon

-- 
nvpublic

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ