From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH 6/6] arm64: tegra: Enable XUSB host in P2972-0000 board Date: Wed, 2 Oct 2019 12:26:11 +0200 Message-ID: <20191002102611.GH3716706@ulmo> References: <20191002080051.11142-1-jckuo@nvidia.com> <20191002080051.11142-7-jckuo@nvidia.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="mYYhpFXgKVw71fwr" Return-path: Content-Disposition: inline In-Reply-To: <20191002080051.11142-7-jckuo@nvidia.com> Sender: linux-kernel-owner@vger.kernel.org To: JC Kuo Cc: gregkh@linuxfoundation.org, jonathanh@nvidia.com, linux-tegra@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, nkristam@nvidia.com, skomatineni@nvidia.com List-Id: devicetree@vger.kernel.org --mYYhpFXgKVw71fwr Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Oct 02, 2019 at 04:00:51PM +0800, JC Kuo wrote: > This commit enables XUSB host and pad controller in Tegra194 > P2972-0000 board. >=20 > Signed-off-by: JC Kuo > --- > .../arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 31 +++++++++- > .../boot/dts/nvidia/tegra194-p2972-0000.dts | 59 +++++++++++++++++++ > 2 files changed, 89 insertions(+), 1 deletion(-) >=20 > diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi b/arch/arm64/= boot/dts/nvidia/tegra194-p2888.dtsi > index 4c38426a6969..cb236edc6a0d 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi > @@ -229,7 +229,7 @@ > regulator-max-microvolt =3D <3300000>; > }; > =20 > - ldo5 { > + vdd_usb_3v3: ldo5 { > regulator-name =3D "VDD_USB_3V3"; > regulator-min-microvolt =3D <3300000>; > regulator-max-microvolt =3D <3300000>; > @@ -313,5 +313,34 @@ > regulator-boot-on; > enable-active-low; > }; > + > + vdd_5v_sata: regulator@4 { > + compatible =3D "regulator-fixed"; > + reg =3D <4>; > + > + regulator-name =3D "vdd-5v-sata"; Please keep capitalization of regulator names consistent. We use all-caps and underscores for the others (which mirrors the names in the schematics), so please stick with that for this one as well. Also, I'm wondering if perhaps you can clarify something here. My understanding is that SATA functionality is provided via a controller on the PCI bus. Why is it that we route the 5 V SATA power to the USB port? Oh wait... this is one of those eSATAp (hybrid) ports that can take either eSATA or USB, right? Do we need any additional setup to switch between eSATA and USB modes? Or is this all done in hardware? That is, if I plug in an eSATA, does it automatically hotplug detect the device as SATA and if I plug in a USB device, does it automatically detect it as USB? > + regulator-min-microvolt =3D <5000000>; > + regulator-max-microvolt =3D <5000000>; > + gpio =3D <&gpio TEGRA194_MAIN_GPIO(Z, 1) GPIO_ACTIVE_LOW>; This will actually cause a warning on boot. For fixed regulators the polarity of the enable GPIO is not specified in the GPIO specifier. Instead you're supposed to use the boolean enable-active-high property to specify if the enable GPIO is active-high. By default the enable GPIO is considered to be active-low. The GPIO specifier needs to have the GPIO_ACTIVE_HIGH flag set regardless for backwards-compatibilitiy reasons. Note that regulator@3 above your new entry does this wrongly, but next-20191002 should have a fix for that. > + }; > + }; > + > + padctl@3520000 { Don't forget to move this into /cbb as well to match the changes to patch 5/6. > + avdd-usb-supply =3D <&vdd_usb_3v3>; > + vclamp-usb-supply =3D <&vdd_1v8ao>; > + ports { Blank line between the above two for better readability. > + usb2-1 { > + vbus-supply =3D <&vdd_5v0_sys>; > + }; > + usb2-3 { Same here and below. > + vbus-supply =3D <&vdd_5v_sata>; > + }; > + usb3-0 { > + vbus-supply =3D <&vdd_5v0_sys>; > + }; > + usb3-3 { > + vbus-supply =3D <&vdd_5v0_sys>; > + }; > + }; > }; > }; > diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts b/arch/ar= m64/boot/dts/nvidia/tegra194-p2972-0000.dts > index d47cd8c4dd24..410221927dfa 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts > +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2972-0000.dts > @@ -222,4 +222,63 @@ > }; > }; > }; > + > + padctl@3520000 { Same comment as above. Move this into /cbb. > + status =3D "okay"; > + > + pads { > + usb2 { > + lanes { > + usb2-1 { > + status =3D "okay"; > + }; > + usb2-2 { And blank lines for readability here and below. > + status =3D "okay"; > + }; > + usb2-3 { > + status =3D "okay"; > + }; > + }; > + }; > + usb3 { > + lanes { > + usb3-0 { > + status =3D "okay"; > + }; > + usb3-3 { > + status =3D "okay"; > + }; > + }; > + }; > + }; > + > + ports { > + usb2-1 { > + mode =3D "host"; > + status =3D "okay"; > + }; > + usb2-3 { > + mode =3D "host"; > + status =3D "okay"; > + }; > + usb3-0 { > + nvidia,usb2-companion =3D <1>; > + status =3D "okay"; > + }; > + usb3-3 { > + nvidia,usb2-companion =3D <3>; > + nvidia,disable-gen2; > + status =3D "okay"; > + }; > + }; > + }; > + > + tegra_xhci: xhci@3610000 { Also needs to move into /cbb. Also, you can drop the tegra_xhci label here since we never reference the controller from elsewhere. Also make sure to update the name here to match the changes in 5/6. Thierry > + status =3D "okay"; > + phys =3D <&{/padctl@3520000/pads/usb2/lanes/usb2-1}>, > + <&{/padctl@3520000/pads/usb2/lanes/usb2-3}>, > + <&{/padctl@3520000/pads/usb3/lanes/usb3-0}>, > + <&{/padctl@3520000/pads/usb3/lanes/usb3-3}>; > + phy-names =3D "usb2-1", "usb2-3", "usb3-0", "usb3-3"; > + }; > }; > --=20 > 2.17.1 >=20 --mYYhpFXgKVw71fwr Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl2Ue0AACgkQ3SOs138+ s6GAgw//bgErm+FyPqbnCyw/CJbKUYXSkpwC/rGhxRnk037h2UYIJSIXtCRFllmx SDCCTvU0l9ii/dSkaVfo/eOpp/B1RTI0E9mdtJkzaFdmedkogE9m+JrlDGBMeXMe WwEksdW08PDcYOjecHBlcpB7+WA4a097WN1zWezOAGrdvNaaKGTSg3A/QSnFDOc9 Wge4KazOfhy6csT+Sjf++A/0uqWrOVhoIZjbdsDjD/zdua6CT96W06YmkK/qZLRU bWn42dGoJLtRCtRFEzHerUq1P5d9k/C29iWduJ/DyM2SeQP6VvwgYO+DY7pOp+Ng sWHN3yyIzR9G0rUpdH+R6KwpFbwqbfNpD4h5U/Z1P/bt8T9TVBdVxrYQnzCRm4ry Mg1FUW13zbmvH5rUgewEdvhGAvRO90SvIQx1Z/RHO3wTbWhkT1jQawiqlYl57fBc bixdkc7Uft4GvSZPc/0mT0IfWWpssVQMZ9J7DHFJpM/wxs8pV84sPJZbg3CSo6C9 PqW917GsFFOCfNjHq1GukmS0C3OKOM4lgwhHCdZ4QaRTK/zzkZohS/iSigSUaxQM 7TC7isxVGkApoeQ8TjugbhW+xeLTVv0PA/ebtnAoBfOQA3vfsB+GIxKEJp69HrbF +Q3D36qqMfE8I1YW0+mCAvH6uEqqYBWDyrEiTvoG0GpZEeQVNrQ= =0eUt -----END PGP SIGNATURE----- --mYYhpFXgKVw71fwr--