All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: phy: tegra-xusb: Convert to json-schema
@ 2022-11-18 14:40 Thierry Reding
  2022-11-30 17:05 ` Rob Herring
  0 siblings, 1 reply; 2+ messages in thread
From: Thierry Reding @ 2022-11-18 14:40 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski; +Cc: Jon Hunter, devicetree, linux-tegra

From: Thierry Reding <treding@nvidia.com>

Convert the Tegra XUSB pad controller bindings from free-form text
format to json-schema.

Signed-off-by: Thierry Reding <treding@nvidia.com>
---
Changes in v2:
- split up into multiple schemas

 .../phy/nvidia,tegra124-xusb-padctl.txt       | 779 --------------
 .../phy/nvidia,tegra124-xusb-padctl.yaml      | 797 +++++++++++++++
 .../phy/nvidia,tegra186-xusb-padctl.yaml      | 632 ++++++++++++
 .../phy/nvidia,tegra194-xusb-padctl.yaml      | 731 ++++++++++++++
 .../phy/nvidia,tegra210-xusb-padctl.yaml      | 946 ++++++++++++++++++
 5 files changed, 3106 insertions(+), 779 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt
 create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
 create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra186-xusb-padctl.yaml
 create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
 create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra210-xusb-padctl.yaml

diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt
deleted file mode 100644
index b62397d2bb0c..000000000000
--- a/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt
+++ /dev/null
@@ -1,779 +0,0 @@
-Device tree binding for NVIDIA Tegra XUSB pad controller
-========================================================
-
-The Tegra XUSB pad controller manages a set of I/O lanes (with differential
-signals) which connect directly to pins/pads on the SoC package. Each lane
-is controlled by a HW block referred to as a "pad" in the Tegra hardware
-documentation. Each such "pad" may control either one or multiple lanes,
-and thus contains any logic common to all its lanes. Each lane can be
-separately configured and powered up.
-
-Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
-super-speed USB. Other lanes are for various types of low-speed, full-speed
-or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
-contains a software-configurable mux that sits between the I/O controller
-ports (e.g. PCIe) and the lanes.
-
-In addition to per-lane configuration, USB 3.0 ports may require additional
-settings on a per-board basis.
-
-Pads will be represented as children of the top-level XUSB pad controller
-device tree node. Each lane exposed by the pad will be represented by its
-own subnode and can be referenced by users of the lane using the standard
-PHY bindings, as described by the phy-bindings.txt file in this directory.
-
-The Tegra hardware documentation refers to the connection between the XUSB
-pad controller and the XUSB controller as "ports". This is confusing since
-"port" is typically used to denote the physical USB receptacle. The device
-tree binding in this document uses the term "port" to refer to the logical
-abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
-for the USB signal, the VBUS power supply, the USB 2.0 companion port for
-USB 3.0 receptacles, ...).
-
-Required properties:
---------------------
-- compatible: Must be:
-  - Tegra124: "nvidia,tegra124-xusb-padctl"
-  - Tegra132: "nvidia,tegra132-xusb-padctl", "nvidia,tegra124-xusb-padctl"
-  - Tegra210: "nvidia,tegra210-xusb-padctl"
-  - Tegra186: "nvidia,tegra186-xusb-padctl"
-  - Tegra194: "nvidia,tegra194-xusb-padctl"
-- reg: Physical base address and length of the controller's registers.
-- resets: Must contain an entry for each entry in reset-names.
-- reset-names: Must include the following entries:
-  - "padctl"
-
-For Tegra124:
-- avdd-pll-utmip-supply: UTMI PLL power supply. Must supply 1.8 V.
-- avdd-pll-erefe-supply: PLLE reference PLL power supply. Must supply 1.05 V.
-- avdd-pex-pll-supply: PCIe/USB3 PLL power supply. Must supply 1.05 V.
-- hvdd-pex-pll-e-supply: High-voltage PLLE power supply. Must supply 3.3 V.
-
-For Tegra210:
-- avdd-pll-utmip-supply: UTMI PLL power supply. Must supply 1.8 V.
-- avdd-pll-uerefe-supply: PLLE reference PLL power supply. Must supply 1.05 V.
-- dvdd-pex-pll-supply: PCIe/USB3 PLL power supply. Must supply 1.05 V.
-- hvdd-pex-pll-e-supply: High-voltage PLLE power supply. Must supply 1.8 V.
-- nvidia,pmc: phandle and specifier referring to the Tegra210 PMC node.
-
-For Tegra186:
-- avdd-pll-erefeut-supply: UPHY brick and reference clock as well as UTMI PHY
-  power supply. Must supply 1.8 V.
-- avdd-usb-supply: USB I/Os, VBUS, ID, REXT, D+/D- power supply. Must supply
-  3.3 V.
-- vclamp-usb-supply: Bias rail for USB pad. Must supply 1.8 V.
-- vddio-hsic-supply: HSIC PHY power supply. Must supply 1.2 V.
-
-For Tegra194:
-- avdd-usb-supply: USB I/Os, VBUS, ID, REXT, D+/D- power supply. Must supply
-  3.3 V.
-- vclamp-usb-supply: Bias rail for USB pad. Must supply 1.8 V.
-
-Pad nodes:
-==========
-
-A required child node named "pads" contains a list of subnodes, one for each
-of the pads exposed by the XUSB pad controller. Each pad may need additional
-resources that can be referenced in its pad node.
-
-The "status" property is used to enable or disable the use of a pad. If set
-to "disabled", the pad will not be used on the given board. In order to use
-the pad and any of its lanes, this property must be set to "okay".
-
-For Tegra124 and Tegra132, the following pads exist: usb2, ulpi, hsic, pcie
-and sata. No extra resources are required for operation of these pads.
-
-For Tegra210, the following pads exist: usb2, hsic, pcie and sata. Below is
-a description of the properties of each pad.
-
-UTMI pad:
----------
-
-Required properties:
-- clocks: Must contain an entry for each entry in clock-names.
-- clock-names: Must contain the following entries:
-  - "trk": phandle and specifier referring to the USB2 tracking clock
-
-HSIC pad:
----------
-
-Required properties:
-- clocks: Must contain an entry for each entry in clock-names.
-- clock-names: Must contain the following entries:
-  - "trk": phandle and specifier referring to the HSIC tracking clock
-
-PCIe pad:
----------
-
-Required properties:
-- clocks: Must contain an entry for each entry in clock-names.
-- clock-names: Must contain the following entries:
-  - "pll": phandle and specifier referring to the PLLE
-- resets: Must contain an entry for each entry in reset-names.
-- reset-names: Must contain the following entries:
-  - "phy": reset for the PCIe UPHY block
-
-SATA pad:
----------
-
-Required properties:
-- resets: Must contain an entry for each entry in reset-names.
-- reset-names: Must contain the following entries:
-  - "phy": reset for the SATA UPHY block
-
-
-PHY nodes:
-==========
-
-Each pad node has a child named "lanes" that contains one or more children of
-its own, each representing one of the lanes controlled by the pad.
-
-Required properties:
---------------------
-- status: Defines the operation status of the PHY. Valid values are:
-  - "disabled": the PHY is disabled
-  - "okay": the PHY is enabled
-- #phy-cells: Should be 0. Since each lane represents a single PHY, there is
-  no need for an additional specifier.
-- nvidia,function: The output function of the PHY. See below for a list of
-  valid functions per SoC generation.
-
-For Tegra124 and Tegra132, the list of valid PHY nodes is given below:
-- usb2: usb2-0, usb2-1, usb2-2
-  - functions: "snps", "xusb", "uart"
-- ulpi: ulpi-0
-  - functions: "snps", "xusb"
-- hsic: hsic-0, hsic-1
-  - functions: "snps", "xusb"
-- pcie: pcie-0, pcie-1, pcie-2, pcie-3, pcie-4
-  - functions: "pcie", "usb3-ss"
-- sata: sata-0
-  - functions: "usb3-ss", "sata"
-
-For Tegra210, the list of valid PHY nodes is given below:
-- usb2: usb2-0, usb2-1, usb2-2, usb2-3
-  - functions: "snps", "xusb", "uart"
-- hsic: hsic-0, hsic-1
-  - functions: "snps", "xusb"
-- pcie: pcie-0, pcie-1, pcie-2, pcie-3, pcie-4, pcie-5, pcie-6
-  - functions: "pcie-x1", "usb3-ss", "pcie-x4"
-- sata: sata-0
-  - functions: "usb3-ss", "sata"
-
-For Tegra194, the list of valid PHY nodes is given below:
-- usb2: usb2-0, usb2-1, usb2-2, usb2-3
-  - functions: "xusb"
-- usb3: usb3-0, usb3-1, usb3-2, usb3-3
-  - functions: "xusb"
-
-Port nodes:
-===========
-
-A required child node named "ports" contains a list of all the ports exposed
-by the XUSB pad controller. Per-port configuration is only required for USB.
-
-USB2 ports:
------------
-
-Required properties:
-- status: Defines the operation status of the port. Valid values are:
-  - "disabled": the port is disabled
-  - "okay": the port is enabled
-- mode: A string that determines the mode in which to run the port. Valid
-  values are:
-  - "host": for USB host mode
-  - "device": for USB device mode
-  - "otg": for USB OTG mode
-
-Required properties for OTG/Peripheral capable USB2 ports:
-- usb-role-switch: Boolean property to indicate that the port support OTG or
-  peripheral mode. If present, the port supports switching between USB host
-  and peripheral roles. Connector should be added as subnode.
-  See usb/usb-conn-gpio.txt.
-
-Optional properties:
-- nvidia,internal: A boolean property whose presence determines that a port
-  is internal. In the absence of this property the port is considered to be
-  external.
-- vbus-supply: phandle to a regulator supplying the VBUS voltage.
-
-ULPI ports:
------------
-
-Optional properties:
-- status: Defines the operation status of the port. Valid values are:
-  - "disabled": the port is disabled
-  - "okay": the port is enabled
-- nvidia,internal: A boolean property whose presence determines that a port
-  is internal. In the absence of this property the port is considered to be
-  external.
-- vbus-supply: phandle to a regulator supplying the VBUS voltage.
-
-HSIC ports:
------------
-
-Required properties:
-- status: Defines the operation status of the port. Valid values are:
-  - "disabled": the port is disabled
-  - "okay": the port is enabled
-
-Optional properties:
-- vbus-supply: phandle to a regulator supplying the VBUS voltage.
-
-Super-speed USB ports:
-----------------------
-
-Required properties:
-- status: Defines the operation status of the port. Valid values are:
-  - "disabled": the port is disabled
-  - "okay": the port is enabled
-- nvidia,usb2-companion: A single cell that specifies the physical port number
-  to map this super-speed USB port to. The range of valid port numbers varies
-  with the SoC generation:
-  - 0-2: for Tegra124 and Tegra132
-  - 0-3: for Tegra210
-
-Optional properties:
-- nvidia,internal: A boolean property whose presence determines that a port
-  is internal. In the absence of this property the port is considered to be
-  external.
-
-- maximum-speed: Only for Tegra194. A string property that specifies maximum
-  supported speed of a usb3 port. Valid values are:
-  - "super-speed-plus": default, the usb3 port supports USB 3.1 Gen 2 speed.
-  - "super-speed": the usb3 port supports USB 3.1 Gen 1 speed only.
-
-For Tegra124 and Tegra132, the XUSB pad controller exposes the following
-ports:
-- 3x USB2: usb2-0, usb2-1, usb2-2
-- 1x ULPI: ulpi-0
-- 2x HSIC: hsic-0, hsic-1
-- 2x super-speed USB: usb3-0, usb3-1
-
-For Tegra210, the XUSB pad controller exposes the following ports:
-- 4x USB2: usb2-0, usb2-1, usb2-2, usb2-3
-- 2x HSIC: hsic-0, hsic-1
-- 4x super-speed USB: usb3-0, usb3-1, usb3-2, usb3-3
-
-For Tegra194, the XUSB pad controller exposes the following ports:
-- 4x USB2: usb2-0, usb2-1, usb2-2, usb2-3
-- 4x super-speed USB: usb3-0, usb3-1, usb3-2, usb3-3
-
-Examples:
-=========
-
-Tegra124 and Tegra132:
-----------------------
-
-SoC include:
-
-	padctl@7009f000 {
-		/* for Tegra124 */
-		compatible = "nvidia,tegra124-xusb-padctl";
-		/* for Tegra132 */
-		compatible = "nvidia,tegra132-xusb-padctl",
-			     "nvidia,tegra124-xusb-padctl";
-		reg = <0x0 0x7009f000 0x0 0x1000>;
-		resets = <&tegra_car 142>;
-		reset-names = "padctl";
-
-		pads {
-			usb2 {
-				status = "disabled";
-
-				lanes {
-					usb2-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					usb2-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					usb2-2 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			ulpi {
-				status = "disabled";
-
-				lanes {
-					ulpi-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			hsic {
-				status = "disabled";
-
-				lanes {
-					hsic-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					hsic-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			pcie {
-				status = "disabled";
-
-				lanes {
-					pcie-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-2 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-3 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-4 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			sata {
-				status = "disabled";
-
-				lanes {
-					sata-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-		};
-
-		ports {
-			usb2-0 {
-				status = "disabled";
-			};
-
-			usb2-1 {
-				status = "disabled";
-			};
-
-			usb2-2 {
-				status = "disabled";
-			};
-
-			ulpi-0 {
-				status = "disabled";
-			};
-
-			hsic-0 {
-				status = "disabled";
-			};
-
-			hsic-1 {
-				status = "disabled";
-			};
-
-			usb3-0 {
-				status = "disabled";
-			};
-
-			usb3-1 {
-				status = "disabled";
-			};
-		};
-	};
-
-Board file:
-
-	padctl@7009f000 {
-		status = "okay";
-
-		pads {
-			usb2 {
-				status = "okay";
-
-				lanes {
-					usb2-0 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-
-					usb2-1 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-
-					usb2-2 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-				};
-			};
-
-			pcie {
-				status = "okay";
-
-				lanes {
-					pcie-0 {
-						nvidia,function = "usb3-ss";
-						status = "okay";
-					};
-
-					pcie-2 {
-						nvidia,function = "pcie";
-						status = "okay";
-					};
-
-					pcie-4 {
-						nvidia,function = "pcie";
-						status = "okay";
-					};
-				};
-			};
-
-			sata {
-				status = "okay";
-
-				lanes {
-					sata-0 {
-						nvidia,function = "sata";
-						status = "okay";
-					};
-				};
-			};
-		};
-
-		ports {
-			/* Micro A/B */
-			usb2-0 {
-				status = "okay";
-				mode = "otg";
-			};
-
-			/* Mini PCIe */
-			usb2-1 {
-				status = "okay";
-				mode = "host";
-			};
-
-			/* USB3 */
-			usb2-2 {
-				status = "okay";
-				mode = "host";
-
-				vbus-supply = <&vdd_usb3_vbus>;
-			};
-
-			usb3-0 {
-				nvidia,port = <2>;
-				status = "okay";
-			};
-		};
-	};
-
-Tegra210:
----------
-
-SoC include:
-
-	padctl@7009f000 {
-		compatible = "nvidia,tegra210-xusb-padctl";
-		reg = <0x0 0x7009f000 0x0 0x1000>;
-		resets = <&tegra_car 142>;
-		reset-names = "padctl";
-
-		status = "disabled";
-
-		pads {
-			usb2 {
-				clocks = <&tegra_car TEGRA210_CLK_USB2_TRK>;
-				clock-names = "trk";
-				status = "disabled";
-
-				lanes {
-					usb2-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					usb2-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					usb2-2 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					usb2-3 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			hsic {
-				clocks = <&tegra_car TEGRA210_CLK_HSIC_TRK>;
-				clock-names = "trk";
-				status = "disabled";
-
-				lanes {
-					hsic-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					hsic-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			pcie {
-				clocks = <&tegra_car TEGRA210_CLK_PLL_E>;
-				clock-names = "pll";
-				resets = <&tegra_car 205>;
-				reset-names = "phy";
-				status = "disabled";
-
-				lanes {
-					pcie-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-1 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-2 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-3 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-4 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-5 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-
-					pcie-6 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-
-			sata {
-				clocks = <&tegra_car TEGRA210_CLK_PLL_E>;
-				clock-names = "pll";
-				resets = <&tegra_car 204>;
-				reset-names = "phy";
-				status = "disabled";
-
-				lanes {
-					sata-0 {
-						status = "disabled";
-						#phy-cells = <0>;
-					};
-				};
-			};
-		};
-
-		ports {
-			usb2-0 {
-				status = "disabled";
-			};
-
-			usb2-1 {
-				status = "disabled";
-			};
-
-			usb2-2 {
-				status = "disabled";
-			};
-
-			usb2-3 {
-				status = "disabled";
-			};
-
-			hsic-0 {
-				status = "disabled";
-			};
-
-			hsic-1 {
-				status = "disabled";
-			};
-
-			usb3-0 {
-				status = "disabled";
-			};
-
-			usb3-1 {
-				status = "disabled";
-			};
-
-			usb3-2 {
-				status = "disabled";
-			};
-
-			usb3-3 {
-				status = "disabled";
-			};
-		};
-	};
-
-Board file:
-
-	padctl@7009f000 {
-		status = "okay";
-
-		pads {
-			usb2 {
-				status = "okay";
-
-				lanes {
-					usb2-0 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-
-					usb2-1 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-
-					usb2-2 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-
-					usb2-3 {
-						nvidia,function = "xusb";
-						status = "okay";
-					};
-				};
-			};
-
-			pcie {
-				status = "okay";
-
-				lanes {
-					pcie-0 {
-						nvidia,function = "pcie-x1";
-						status = "okay";
-					};
-
-					pcie-1 {
-						nvidia,function = "pcie-x4";
-						status = "okay";
-					};
-
-					pcie-2 {
-						nvidia,function = "pcie-x4";
-						status = "okay";
-					};
-
-					pcie-3 {
-						nvidia,function = "pcie-x4";
-						status = "okay";
-					};
-
-					pcie-4 {
-						nvidia,function = "pcie-x4";
-						status = "okay";
-					};
-
-					pcie-5 {
-						nvidia,function = "usb3-ss";
-						status = "okay";
-					};
-
-					pcie-6 {
-						nvidia,function = "usb3-ss";
-						status = "okay";
-					};
-				};
-			};
-
-			sata {
-				status = "okay";
-
-				lanes {
-					sata-0 {
-						nvidia,function = "sata";
-						status = "okay";
-					};
-				};
-			};
-		};
-
-		ports {
-			usb2-0 {
-				status = "okay";
-				mode = "otg";
-			};
-
-			usb2-1 {
-				status = "okay";
-				vbus-supply = <&vdd_5v0_rtl>;
-				mode = "host";
-			};
-
-			usb2-2 {
-				status = "okay";
-				vbus-supply = <&vdd_usb_vbus>;
-				mode = "host";
-			};
-
-			usb2-3 {
-				status = "okay";
-				mode = "host";
-			};
-
-			usb3-0 {
-				status = "okay";
-				nvidia,lanes = "pcie-6";
-				nvidia,port = <1>;
-			};
-
-			usb3-1 {
-				status = "okay";
-				nvidia,lanes = "pcie-5";
-				nvidia,port = <2>;
-			};
-		};
-	};
diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
new file mode 100644
index 000000000000..f1121a340a72
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
@@ -0,0 +1,797 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/nvidia,tegra124-xusb-padctl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NVIDIA Tegra124 XUSB pad controller
+
+maintainers:
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Jon Hunter <jonathanh@nvidia.com>
+
+description:
+  The Tegra XUSB pad controller manages a set of I/O lanes (with differential
+  signals) which connect directly to pins/pads on the SoC package. Each lane
+  is controlled by a HW block referred to as a "pad" in the Tegra hardware
+  documentation. Each such "pad" may control either one or multiple lanes,
+  and thus contains any logic common to all its lanes. Each lane can be
+  separately configured and powered up.
+
+  Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
+  super-speed USB. Other lanes are for various types of low-speed, full-speed
+  or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
+  contains a software-configurable mux that sits between the I/O controller
+  ports (e.g. PCIe) and the lanes.
+
+  In addition to per-lane configuration, USB 3.0 ports may require additional
+  settings on a per-board basis.
+
+  Pads will be represented as children of the top-level XUSB pad controller
+  device tree node. Each lane exposed by the pad will be represented by its
+  own subnode and can be referenced by users of the lane using the standard
+  PHY bindings, as described by the phy-bindings.txt file in this directory.
+
+  The Tegra hardware documentation refers to the connection between the XUSB
+  pad controller and the XUSB controller as "ports". This is confusing since
+  "port" is typically used to denote the physical USB receptacle. The device
+  tree binding in this document uses the term "port" to refer to the logical
+  abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
+  for the USB signal, the VBUS power supply, the USB 2.0 companion port for
+  USB 3.0 receptacles, ...).
+
+properties:
+  compatible:
+    oneOf:
+      - enum:
+          - nvidia,tegra124-xusb-padctl
+
+      - items:
+          - const: nvidia,tegra132-xusb-padctl
+          - const: nvidia,tegra124-xusb-padctl
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    items:
+      - description: XUSB pad controller interrupt
+
+  resets:
+    items:
+      - description: pad controller reset
+
+  reset-names:
+    items:
+      - const: padctl
+
+  avdd-pll-utmip-supply:
+    description: UTMI PLL power supply. Must supply 1.8 V.
+
+  avdd-pll-erefe-supply:
+    description: PLLE reference PLL power supply. Must supply 1.05 V.
+
+  avdd-pex-pll-supply:
+    description: PCIe/USB3 PLL power supply. Must supply 1.05 V.
+
+  hvdd-pex-pll-e-supply:
+    description: High-voltage PLLE power supply. Must supply 3.3 V.
+
+  pads:
+    description: A required child node named "pads" contains a list of
+      subnodes, one for each of the pads exposed by the XUSB pad controller.
+      Each pad may need additional resources that can be referenced in its
+      pad node.
+
+      The "status" property is used to enable or disable the use of a pad.
+      If set to "disabled", the pad will not be used on the given board. In
+      order to use the pad and any of its lanes, this property must be set
+      to "okay".
+    type: object
+    properties:
+      usb2:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: USB2 tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              usb2-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      ulpi:
+        type: object
+        properties:
+          lanes:
+            type: object
+            properties:
+              ulpi-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      hsic:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: HSIC tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              hsic-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              hsic-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      pcie:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: PLLE clock
+
+          clock-names:
+            items:
+              - const: pll
+
+          resets:
+            items:
+              - description: reset for the PCIe UPHY block
+
+          reset-names:
+            items:
+              - const: phy
+
+          lanes:
+            type: object
+            properties:
+              pcie-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-3:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-4:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      sata:
+        type: object
+        properties:
+          resets:
+            items:
+              - description: reset for the SATA UPHY block
+
+          reset-names:
+            items:
+              - const: phy
+
+          lanes:
+            type: object
+            properties:
+              sata-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ sata, usb3-ss ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+  ports:
+    description: A required child node named "ports" contains a list of
+      subnodes, one for each of the ports exposed by the XUSB pad controller.
+      Each port may need additional resources that can be referenced in its
+      port node.
+
+      The "status" property is used to enable or disable the use of a port.
+      If set to "disabled", the port will not be used on the given board. In
+      order to use the port, this property must be set to "okay".
+    type: object
+    properties:
+      usb2-0:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb2-1:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb2-2:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      ulpi-0:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      hsic-0:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      hsic-1:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-0:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-1:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - resets
+  - reset-names
+  - avdd-pll-utmip-supply
+  - avdd-pll-erefe-supply
+  - avdd-pex-pll-supply
+  - hvdd-pex-pll-e-supply
+
+examples:
+  # Tegra124 and Tegra132
+  - |
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+    padctl@7009f000 {
+        compatible = "nvidia,tegra124-xusb-padctl";
+        reg = <0x7009f000 0x1000>;
+        interrupts = <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>;
+        resets = <&tegra_car 142>;
+        reset-names = "padctl";
+
+        avdd-pll-utmip-supply = <&vddio_1v8>;
+        avdd-pll-erefe-supply = <&avdd_1v05_run>;
+        avdd-pex-pll-supply = <&vdd_1v05_run>;
+        hvdd-pex-pll-e-supply = <&vdd_3v3_lp0>;
+
+        pads {
+            usb2 {
+                lanes {
+                    usb2-0 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-1 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-2 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            ulpi {
+                lanes {
+                    ulpi-0 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            hsic {
+                lanes {
+                    hsic-0 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    hsic-1 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            pcie {
+                lanes {
+                    pcie-0 {
+                        nvidia,function = "usb3-ss";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-1 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-2 {
+                        nvidia,function = "pcie";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-3 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-4 {
+                        nvidia,function = "pcie";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            sata {
+                lanes {
+                    sata-0 {
+                        nvidia,function = "sata";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+        };
+
+        ports {
+            /* Micro A/B */
+            usb2-0 {
+                mode = "otg";
+            };
+
+            /* Mini PCIe */
+            usb2-1 {
+                mode = "host";
+            };
+
+            /* USB3 */
+            usb2-2 {
+                vbus-supply = <&vdd_usb3_vbus>;
+                mode = "host";
+            };
+
+            ulpi-0 {
+                status = "disabled";
+            };
+
+            hsic-0 {
+                status = "disabled";
+            };
+
+            hsic-1 {
+                status = "disabled";
+            };
+
+            usb3-0 {
+                nvidia,usb2-companion = <2>;
+            };
+
+            usb3-1 {
+                status = "disabled";
+            };
+        };
+    };
diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra186-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra186-xusb-padctl.yaml
new file mode 100644
index 000000000000..e202e296a8b1
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/nvidia,tegra186-xusb-padctl.yaml
@@ -0,0 +1,632 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/nvidia,tegra186-xusb-padctl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NVIDIA Tegra186 XUSB pad controller
+
+maintainers:
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Jon Hunter <jonathanh@nvidia.com>
+
+description:
+  The Tegra XUSB pad controller manages a set of I/O lanes (with differential
+  signals) which connect directly to pins/pads on the SoC package. Each lane
+  is controlled by a HW block referred to as a "pad" in the Tegra hardware
+  documentation. Each such "pad" may control either one or multiple lanes,
+  and thus contains any logic common to all its lanes. Each lane can be
+  separately configured and powered up.
+
+  Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
+  super-speed USB. Other lanes are for various types of low-speed, full-speed
+  or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
+  contains a software-configurable mux that sits between the I/O controller
+  ports (e.g. PCIe) and the lanes.
+
+  In addition to per-lane configuration, USB 3.0 ports may require additional
+  settings on a per-board basis.
+
+  Pads will be represented as children of the top-level XUSB pad controller
+  device tree node. Each lane exposed by the pad will be represented by its
+  own subnode and can be referenced by users of the lane using the standard
+  PHY bindings, as described by the phy-bindings.txt file in this directory.
+
+  The Tegra hardware documentation refers to the connection between the XUSB
+  pad controller and the XUSB controller as "ports". This is confusing since
+  "port" is typically used to denote the physical USB receptacle. The device
+  tree binding in this document uses the term "port" to refer to the logical
+  abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
+  for the USB signal, the VBUS power supply, the USB 2.0 companion port for
+  USB 3.0 receptacles, ...).
+
+properties:
+  compatible:
+    const: nvidia,tegra186-xusb-padctl
+
+  reg:
+    items:
+      - description: pad controller registers
+      - description: AO registers
+
+  interrupts:
+    items:
+      - description: XUSB pad controller interrupt
+
+  reg-names:
+    items:
+      - const: padctl
+      - const: ao
+
+  resets:
+    items:
+      - description: pad controller reset
+
+  reset-names:
+    items:
+      - const: padctl
+
+  avdd-pll-erefeut-supply:
+    description: UPHY brick and reference clock as well as UTMI PHY
+      power supply. Must supply 1.8 V.
+
+  avdd-usb-supply:
+    description: USB I/Os, VBUS, ID, REXT, D+/D- power supply. Must
+      supply 3.3 V.
+
+  vclamp-usb-supply:
+    description: Bias rail for USB pad. Must supply 1.8 V.
+
+  vddio-hsic-supply:
+    description: HSIC PHY power supply. Must supply 1.2 V.
+
+  pads:
+    description: A required child node named "pads" contains a list of
+      subnodes, one for each of the pads exposed by the XUSB pad controller.
+      Each pad may need additional resources that can be referenced in its
+      pad node.
+
+      The "status" property is used to enable or disable the use of a pad.
+      If set to "disabled", the pad will not be used on the given board. In
+      order to use the pad and any of its lanes, this property must be set
+      to "okay".
+    type: object
+    properties:
+      usb2:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: USB2 tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              usb2-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      hsic:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: HSIC tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              hsic-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      usb3:
+        type: object
+        properties:
+          lanes:
+            type: object
+            properties:
+              usb3-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb3-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb3-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+  ports:
+    description: A required child node named "ports" contains a list of
+      subnodes, one for each of the ports exposed by the XUSB pad controller.
+      Each port may need additional resources that can be referenced in its
+      port node.
+
+      The "status" property is used to enable or disable the use of a port.
+      If set to "disabled", the port will not be used on the given board. In
+      order to use the port, this property must be set to "okay".
+    type: object
+    properties:
+      usb2-0:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-1:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-2:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      hsic-0:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      usb3-0:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+      usb3-1:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+      usb3-2:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+    additionalProperties: false
+
+additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - resets
+  - reset-names
+  - avdd-pll-erefeut-supply
+  - avdd-usb-supply
+  - vclamp-usb-supply
+  - vddio-hsic-supply
+
+examples:
+  - |
+    #include <dt-bindings/clock/tegra186-clock.h>
+    #include <dt-bindings/gpio/tegra186-gpio.h>
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+    #include <dt-bindings/reset/tegra186-reset.h>
+
+    padctl@3520000 {
+        compatible = "nvidia,tegra186-xusb-padctl";
+        reg = <0x03520000 0x1000>,
+              <0x03540000 0x1000>;
+        reg-names = "padctl", "ao";
+        interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>;
+
+        resets = <&bpmp TEGRA186_RESET_XUSB_PADCTL>;
+        reset-names = "padctl";
+
+        avdd-pll-erefeut-supply = <&vdd_1v8_pll>;
+        avdd-usb-supply = <&vdd_3v3_sys>;
+        vclamp-usb-supply = <&vdd_1v8>;
+        vddio-hsic-supply = <&gnd>;
+
+        pads {
+            usb2 {
+                clocks = <&bpmp TEGRA186_CLK_USB2_TRK>;
+                clock-names = "trk";
+
+                lanes {
+                    usb2-0 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-1 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-2 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            hsic {
+                clocks = <&bpmp TEGRA186_CLK_HSIC_TRK>;
+                clock-names = "trk";
+                status = "disabled";
+
+                lanes {
+                    hsic-0 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            usb3 {
+                lanes {
+                    usb3-0 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb3-1 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb3-2 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+        };
+
+        ports {
+            usb2-0 {
+                mode = "otg";
+                vbus-supply = <&vdd_usb0>;
+                usb-role-switch;
+
+                connector {
+                    compatible = "gpio-usb-b-connector",
+                                 "usb-b-connector";
+                    label = "micro-USB";
+                    type = "micro";
+                    vbus-gpios = <&gpio TEGRA186_MAIN_GPIO(X, 7) GPIO_ACTIVE_LOW>;
+                    id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
+                };
+            };
+
+            usb2-1 {
+                vbus-supply = <&vdd_usb1>;
+                mode = "host";
+            };
+
+            usb2-2 {
+                status = "disabled";
+            };
+
+            hsic-0 {
+                status = "disabled";
+            };
+
+            usb3-0 {
+                nvidia,usb2-companion = <1>;
+            };
+
+            usb3-1 {
+                status = "disabled";
+            };
+
+            usb3-2 {
+                status = "disabled";
+            };
+        };
+    };
diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
new file mode 100644
index 000000000000..1c570ff65eb4
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
@@ -0,0 +1,731 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/nvidia,tegra194-xusb-padctl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NVIDIA Tegra194 XUSB pad controller
+
+maintainers:
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Jon Hunter <jonathanh@nvidia.com>
+
+description:
+  The Tegra XUSB pad controller manages a set of I/O lanes (with differential
+  signals) which connect directly to pins/pads on the SoC package. Each lane
+  is controlled by a HW block referred to as a "pad" in the Tegra hardware
+  documentation. Each such "pad" may control either one or multiple lanes,
+  and thus contains any logic common to all its lanes. Each lane can be
+  separately configured and powered up.
+
+  Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
+  super-speed USB. Other lanes are for various types of low-speed, full-speed
+  or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
+  contains a software-configurable mux that sits between the I/O controller
+  ports (e.g. PCIe) and the lanes.
+
+  In addition to per-lane configuration, USB 3.0 ports may require additional
+  settings on a per-board basis.
+
+  Pads will be represented as children of the top-level XUSB pad controller
+  device tree node. Each lane exposed by the pad will be represented by its
+  own subnode and can be referenced by users of the lane using the standard
+  PHY bindings, as described by the phy-bindings.txt file in this directory.
+
+  The Tegra hardware documentation refers to the connection between the XUSB
+  pad controller and the XUSB controller as "ports". This is confusing since
+  "port" is typically used to denote the physical USB receptacle. The device
+  tree binding in this document uses the term "port" to refer to the logical
+  abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
+  for the USB signal, the VBUS power supply, the USB 2.0 companion port for
+  USB 3.0 receptacles, ...).
+
+properties:
+  compatible:
+    const: nvidia,tegra194-xusb-padctl
+
+  reg:
+    items:
+      - description: pad controller registers
+      - description: AO registers
+
+  reg-names:
+    items:
+      - const: padctl
+      - const: ao
+
+  interrupts:
+    items:
+      - description: XUSB pad controller interrupt
+
+  resets:
+    items:
+      - description: pad controller reset
+
+  reset-names:
+    items:
+      - const: padctl
+
+  avdd-usb-supply:
+    description: USB I/Os, VBUS, ID, REXT, D+/D- power supply. Must
+      supply 3.3 V.
+
+  vclamp-usb-supply:
+    description: Bias rail for USB pad. Must supply 1.8 V.
+
+  pads:
+    description: A required child node named "pads" contains a list of
+      subnodes, one for each of the pads exposed by the XUSB pad controller.
+      Each pad may need additional resources that can be referenced in its
+      pad node.
+
+      The "status" property is used to enable or disable the use of a pad.
+      If set to "disabled", the pad will not be used on the given board. In
+      order to use the pad and any of its lanes, this property must be set
+      to "okay".
+    type: object
+    properties:
+      usb2:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: USB2 tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              usb2-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+
+              usb2-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-3:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      usb3:
+        type: object
+        properties:
+          lanes:
+            type: object
+            properties:
+              usb3-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb3-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb3-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb3-3:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+  ports:
+    description: A required child node named "ports" contains a list of
+      subnodes, one for each of the ports exposed by the XUSB pad controller.
+      Each port may need additional resources that can be referenced in its
+      port node.
+
+      The "status" property is used to enable or disable the use of a port.
+      If set to "disabled", the port will not be used on the given board. In
+      order to use the port, this property must be set to "okay".
+    type: object
+    properties:
+      usb2-0:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-1:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-2:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-3:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb3-0:
+        type: object
+        properties:
+          maximum-speed:
+            description: A string property that specifies the maximum
+              supported speed of a USB3 port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            oneOf:
+              - description: The USB3 port supports USB 3.1 Gen 2 speed.
+                  This is the default.
+                const: super-speed-plus
+              - description: The USB3 port supports USB 3.1 Gen 1 speed
+                  only.
+                const: super-speed
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-1:
+        type: object
+        properties:
+          maximum-speed:
+            description: A string property that specifies the maximum
+              supported speed of a USB3 port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            oneOf:
+              - description: The USB3 port supports USB 3.1 Gen 2 speed.
+                  This is the default.
+                const: super-speed-plus
+              - description: The USB3 port supports USB 3.1 Gen 1 speed
+                  only.
+                const: super-speed
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-2:
+        type: object
+        properties:
+          maximum-speed:
+            description: A string property that specifies the maximum
+              supported speed of a USB3 port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            oneOf:
+              - description: The USB3 port supports USB 3.1 Gen 2 speed.
+                  This is the default.
+                const: super-speed-plus
+              - description: The USB3 port supports USB 3.1 Gen 1 speed
+                  only.
+                const: super-speed
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-3:
+        type: object
+        properties:
+          maximum-speed:
+            description: A string property that specifies the maximum
+              supported speed of a USB3 port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            oneOf:
+              - description: The USB3 port supports USB 3.1 Gen 2 speed.
+                  This is the default.
+                const: super-speed-plus
+              - description: The USB3 port supports USB 3.1 Gen 1 speed
+                  only.
+                const: super-speed
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - resets
+  - reset-names
+  - avdd-usb-supply
+  - vclamp-usb-supply
+
+examples:
+  - |
+    #include <dt-bindings/clock/tegra194-clock.h>
+    #include <dt-bindings/gpio/tegra194-gpio.h>
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+    #include <dt-bindings/reset/tegra194-reset.h>
+
+    padctl@3520000 {
+        compatible = "nvidia,tegra194-xusb-padctl";
+        reg = <0x03520000 0x1000>,
+              <0x03540000 0x1000>;
+        reg-names = "padctl", "ao";
+        interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>;
+
+        resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>;
+        reset-names = "padctl";
+
+        avdd-usb-supply = <&vdd_usb_3v3>;
+        vclamp-usb-supply = <&vdd_1v8ao>;
+
+        pads {
+            usb2 {
+                clocks = <&bpmp TEGRA194_CLK_USB2_TRK>;
+                clock-names = "trk";
+
+                lanes {
+                    usb2-0 {
+                        nvidia,function = "xusb";
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-1 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-2 {
+                        nvidia,function = "xusb";
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-3 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            usb3 {
+                lanes {
+                    usb3-0 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb3-1 {
+                        nvidia,function = "xusb";
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    usb3-2 {
+                        nvidia,function = "xusb";
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    usb3-3 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+        };
+
+        ports {
+            usb2-0 {
+                status = "disabled";
+            };
+
+            usb2-1 {
+                vbus-supply = <&vdd_5v0_sys>;
+                mode = "host";
+            };
+
+            usb2-2 {
+                status = "disabled";
+            };
+
+            usb2-3 {
+                vbus-supply = <&vdd_5v_sata>;
+                mode = "host";
+            };
+
+            usb3-0 {
+                vbus-supply = <&vdd_5v0_sys>;
+                nvidia,usb2-companion = <1>;
+            };
+
+            usb3-1 {
+                status = "disabled";
+            };
+
+            usb3-2 {
+                status = "disabled";
+            };
+
+            usb3-3 {
+                maximum-speed = "super-speed";
+                vbus-supply = <&vdd_5v0_sys>;
+                nvidia,usb2-companion = <3>;
+            };
+        };
+    };
diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra210-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra210-xusb-padctl.yaml
new file mode 100644
index 000000000000..fd2178e28665
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/nvidia,tegra210-xusb-padctl.yaml
@@ -0,0 +1,946 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/nvidia,tegra210-xusb-padctl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NVIDIA Tegra210 XUSB pad controller
+
+maintainers:
+  - Thierry Reding <thierry.reding@gmail.com>
+  - Jon Hunter <jonathanh@nvidia.com>
+
+description:
+  The Tegra XUSB pad controller manages a set of I/O lanes (with differential
+  signals) which connect directly to pins/pads on the SoC package. Each lane
+  is controlled by a HW block referred to as a "pad" in the Tegra hardware
+  documentation. Each such "pad" may control either one or multiple lanes,
+  and thus contains any logic common to all its lanes. Each lane can be
+  separately configured and powered up.
+
+  Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
+  super-speed USB. Other lanes are for various types of low-speed, full-speed
+  or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
+  contains a software-configurable mux that sits between the I/O controller
+  ports (e.g. PCIe) and the lanes.
+
+  In addition to per-lane configuration, USB 3.0 ports may require additional
+  settings on a per-board basis.
+
+  Pads will be represented as children of the top-level XUSB pad controller
+  device tree node. Each lane exposed by the pad will be represented by its
+  own subnode and can be referenced by users of the lane using the standard
+  PHY bindings, as described by the phy-bindings.txt file in this directory.
+
+  The Tegra hardware documentation refers to the connection between the XUSB
+  pad controller and the XUSB controller as "ports". This is confusing since
+  "port" is typically used to denote the physical USB receptacle. The device
+  tree binding in this document uses the term "port" to refer to the logical
+  abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
+  for the USB signal, the VBUS power supply, the USB 2.0 companion port for
+  USB 3.0 receptacles, ...).
+
+properties:
+  compatible:
+    const: nvidia,tegra210-xusb-padctl
+
+  reg:
+    maxItems: 1
+
+  resets:
+    items:
+      - description: pad controller reset
+
+  interrupts:
+    items:
+      - description: XUSB pad controller interrupt
+
+  reset-names:
+    items:
+      - const: padctl
+
+  avdd-pll-utmip-supply:
+    description: UTMI PLL power supply. Must supply 1.8 V.
+
+  avdd-pll-uerefe-supply:
+    description: PLLE reference PLL power supply. Must supply 1.05 V.
+
+  dvdd-pex-pll-supply:
+    description: PCIe/USB3 PLL power supply. Must supply 1.05 V.
+
+  hvdd-pex-pll-e-supply:
+    description: High-voltage PLLE power supply. Must supply 1.8 V.
+
+  nvidia,pmc:
+    description: phandle to the Tegra Power Management Controller (PMC) node
+    $ref: /schemas/types.yaml#/definitions/phandle
+
+  pads:
+    description: A required child node named "pads" contains a list of
+      subnodes, one for each of the pads exposed by the XUSB pad controller.
+      Each pad may need additional resources that can be referenced in its
+      pad node.
+
+      The "status" property is used to enable or disable the use of a pad.
+      If set to "disabled", the pad will not be used on the given board. In
+      order to use the pad and any of its lanes, this property must be set
+      to "okay".
+    type: object
+    properties:
+      usb2:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: USB2 tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              usb2-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              usb2-3:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb, uart ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      hsic:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: HSIC tracking clock
+
+          clock-names:
+            items:
+              - const: trk
+
+          lanes:
+            type: object
+            properties:
+              hsic-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              hsic-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ snps, xusb ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      pcie:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: PCIe PLL clock source
+
+          clock-names:
+            items:
+              - const: pll
+
+          resets:
+            items:
+              - description: PCIe PHY reset
+
+          reset-names:
+            items:
+              - const: phy
+
+          lanes:
+            type: object
+            properties:
+              pcie-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-1:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-2:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-3:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-4:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-5:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+              pcie-6:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ pcie-x1, usb3-ss, pcie-x4 ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+      sata:
+        type: object
+        properties:
+          clocks:
+            items:
+              - description: SATA PLL clock source
+
+          clock-names:
+            items:
+              - const: pll
+
+          resets:
+            items:
+              - description: SATA PHY reset
+
+          reset-names:
+            items:
+              - const: phy
+
+          lanes:
+            type: object
+            properties:
+              sata-0:
+                type: object
+                properties:
+                  "#phy-cells":
+                    const: 0
+
+                  nvidia,function:
+                    description: Function selection for this lane.
+                    $ref: "/schemas/types.yaml#/definitions/string"
+                    enum: [ usb3-ss, sata ]
+
+                  phandle:
+                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
+
+                  status:
+                    $ref: "/schemas/dt-core.yaml#/properties/status"
+
+                additionalProperties: false
+
+            additionalProperties: false
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+  ports:
+    description: A required child node named "ports" contains a list of
+      subnodes, one for each of the ports exposed by the XUSB pad controller.
+      Each port may need additional resources that can be referenced in its
+      port node.
+
+      The "status" property is used to enable or disable the use of a port.
+      If set to "disabled", the port will not be used on the given board. In
+      order to use the port, this property must be set to "okay".
+    type: object
+    properties:
+      usb2-0:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-1:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-2:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      usb2-3:
+        type: object
+        properties:
+          # no need to further describe this because the connector will
+          # match on gpio-usb-b-connector or usb-b-connector and cause
+          # that binding to be selected for the subnode
+          connector:
+            type: object
+
+          mode:
+            description: A string that determines the mode in which to
+              run the port.
+            $ref: "/schemas/types.yaml#/definitions/string"
+            enum: [ host, peripheral, otg ]
+
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          usb-role-switch:
+            description: |
+              A boolean property whole presence indicates that the port
+              supports OTG or peripheral mode. If present, the port
+              supports switching between USB host and peripheral roles.
+              A connector must be added as a subnode in that case.
+
+              See ../connector/usb-connector.yaml.
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        dependencies:
+          usb-role-switch: [ connector ]
+
+        additionalProperties: false
+
+      hsic-0:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      hsic-1:
+        type: object
+        properties:
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-0:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-1:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-2:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+      usb3-3:
+        type: object
+        properties:
+          nvidia,internal:
+            description: A boolean property whose presence determines
+              that a port is internal. In the absence of this property
+              the port is considered to be external.
+            $ref: "/schemas/types.yaml#/definitions/flag"
+
+          nvidia,usb2-companion:
+            description: A single cell that specifies the physical port
+              number to map this super-speed USB port to. The range of
+              valid port numbers varies with the SoC generation.
+            $ref: "/schemas/types.yaml#/definitions/uint32"
+            enum: [ 0, 1, 2, 3 ]
+
+          status:
+            $ref: "/schemas/dt-core.yaml#/properties/status"
+
+          vbus-supply:
+            description: A phandle to the regulator supplying the VBUS
+              voltage.
+
+        additionalProperties: false
+
+    additionalProperties: false
+
+additionalProperties: false
+
+required:
+  - avdd-pll-utmip-supply
+  - avdd-pll-uerefe-supply
+  - dvdd-pex-pll-supply
+  - hvdd-pex-pll-e-supply
+
+examples:
+  - |
+    #include <dt-bindings/clock/tegra210-car.h>
+    #include <dt-bindings/gpio/tegra-gpio.h>
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+    padctl@7009f000 {
+        compatible = "nvidia,tegra210-xusb-padctl";
+        reg = <0x7009f000 0x1000>;
+        interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>;
+        resets = <&tegra_car 142>;
+        reset-names = "padctl";
+
+        avdd-pll-utmip-supply = <&vdd_1v8>;
+        avdd-pll-uerefe-supply = <&vdd_pex_1v05>;
+        dvdd-pex-pll-supply = <&vdd_pex_1v05>;
+        hvdd-pex-pll-e-supply = <&vdd_1v8>;
+
+        pads {
+            usb2 {
+                clocks = <&tegra_car TEGRA210_CLK_USB2_TRK>;
+                clock-names = "trk";
+
+                lanes {
+                    usb2-0 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-1 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-2 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+
+                    usb2-3 {
+                        nvidia,function = "xusb";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            hsic {
+                clocks = <&tegra_car TEGRA210_CLK_HSIC_TRK>;
+                clock-names = "trk";
+                status = "disabled";
+
+                lanes {
+                    hsic-0 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+
+                    hsic-1 {
+                        status = "disabled";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            pcie {
+                clocks = <&tegra_car TEGRA210_CLK_PLL_E>;
+                clock-names = "pll";
+                resets = <&tegra_car 205>;
+                reset-names = "phy";
+
+                lanes {
+                    pcie-0 {
+                        nvidia,function = "pcie-x1";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-1 {
+                        nvidia,function = "pcie-x4";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-2 {
+                        nvidia,function = "pcie-x4";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-3 {
+                        nvidia,function = "pcie-x4";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-4 {
+                        nvidia,function = "pcie-x4";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-5 {
+                        nvidia,function = "usb3-ss";
+                        #phy-cells = <0>;
+                    };
+
+                    pcie-6 {
+                        nvidia,function = "usb3-ss";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+
+            sata {
+                clocks = <&tegra_car TEGRA210_CLK_PLL_E>;
+                clock-names = "pll";
+                resets = <&tegra_car 204>;
+                reset-names = "phy";
+
+                lanes {
+                    sata-0 {
+                        nvidia,function = "sata";
+                        #phy-cells = <0>;
+                    };
+                };
+            };
+        };
+
+        ports {
+            usb2-0 {
+                mode = "peripheral";
+                usb-role-switch;
+
+                connector {
+                    compatible = "gpio-usb-b-connector",
+                                 "usb-b-connector";
+                    label = "micro-USB";
+                    type = "micro";
+                    vbus-gpios = <&gpio TEGRA_GPIO(CC, 4) GPIO_ACTIVE_LOW>;
+                };
+            };
+
+            usb2-1 {
+                vbus-supply = <&vdd_5v0_rtl>;
+                mode = "host";
+            };
+
+            usb2-2 {
+                vbus-supply = <&vdd_usb_vbus>;
+                mode = "host";
+            };
+
+            usb2-3 {
+                mode = "host";
+            };
+
+            hsic-0 {
+                status = "disabled";
+            };
+
+            hsic-1 {
+                status = "disabled";
+            };
+
+            usb3-0 {
+                nvidia,usb2-companion = <1>;
+            };
+
+            usb3-1 {
+                nvidia,usb2-companion = <2>;
+            };
+
+            usb3-2 {
+                status = "disabled";
+            };
+
+            usb3-3 {
+                status = "disabled";
+            };
+        };
+    };
-- 
2.38.1


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

* Re: [PATCH v2] dt-bindings: phy: tegra-xusb: Convert to json-schema
  2022-11-18 14:40 [PATCH v2] dt-bindings: phy: tegra-xusb: Convert to json-schema Thierry Reding
@ 2022-11-30 17:05 ` Rob Herring
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2022-11-30 17:05 UTC (permalink / raw)
  To: Thierry Reding; +Cc: Krzysztof Kozlowski, Jon Hunter, devicetree, linux-tegra

On Fri, Nov 18, 2022 at 03:40:15PM +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> Convert the Tegra XUSB pad controller bindings from free-form text
> format to json-schema.
> 
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> Changes in v2:
> - split up into multiple schemas
> 
>  .../phy/nvidia,tegra124-xusb-padctl.txt       | 779 --------------
>  .../phy/nvidia,tegra124-xusb-padctl.yaml      | 797 +++++++++++++++
>  .../phy/nvidia,tegra186-xusb-padctl.yaml      | 632 ++++++++++++
>  .../phy/nvidia,tegra194-xusb-padctl.yaml      | 731 ++++++++++++++
>  .../phy/nvidia,tegra210-xusb-padctl.yaml      | 946 ++++++++++++++++++
>  5 files changed, 3106 insertions(+), 779 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt
>  create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
>  create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra186-xusb-padctl.yaml
>  create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
>  create mode 100644 Documentation/devicetree/bindings/phy/nvidia,tegra210-xusb-padctl.yaml

Need to CC the phy maintainers.

[...]

> diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
> new file mode 100644
> index 000000000000..f1121a340a72
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.yaml
> @@ -0,0 +1,797 @@
> +# SPDX-License-Identifier: GPL-2.0-only

All NVIDIA authors on the original, please dual license.

> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/phy/nvidia,tegra124-xusb-padctl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: NVIDIA Tegra124 XUSB pad controller
> +
> +maintainers:
> +  - Thierry Reding <thierry.reding@gmail.com>
> +  - Jon Hunter <jonathanh@nvidia.com>
> +
> +description:

Need a '|'  to maintain paragraphs here.

> +  The Tegra XUSB pad controller manages a set of I/O lanes (with differential
> +  signals) which connect directly to pins/pads on the SoC package. Each lane
> +  is controlled by a HW block referred to as a "pad" in the Tegra hardware
> +  documentation. Each such "pad" may control either one or multiple lanes,
> +  and thus contains any logic common to all its lanes. Each lane can be
> +  separately configured and powered up.
> +
> +  Some of the lanes are high-speed lanes, which can be used for PCIe, SATA or
> +  super-speed USB. Other lanes are for various types of low-speed, full-speed
> +  or high-speed USB (such as UTMI, ULPI and HSIC). The XUSB pad controller
> +  contains a software-configurable mux that sits between the I/O controller
> +  ports (e.g. PCIe) and the lanes.
> +
> +  In addition to per-lane configuration, USB 3.0 ports may require additional
> +  settings on a per-board basis.
> +
> +  Pads will be represented as children of the top-level XUSB pad controller
> +  device tree node. Each lane exposed by the pad will be represented by its
> +  own subnode and can be referenced by users of the lane using the standard
> +  PHY bindings, as described by the phy-bindings.txt file in this directory.
> +
> +  The Tegra hardware documentation refers to the connection between the XUSB
> +  pad controller and the XUSB controller as "ports". This is confusing since
> +  "port" is typically used to denote the physical USB receptacle. The device
> +  tree binding in this document uses the term "port" to refer to the logical
> +  abstraction of the signals that are routed to a USB receptacle (i.e. a PHY
> +  for the USB signal, the VBUS power supply, the USB 2.0 companion port for
> +  USB 3.0 receptacles, ...).
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - enum:
> +          - nvidia,tegra124-xusb-padctl
> +
> +      - items:
> +          - const: nvidia,tegra132-xusb-padctl
> +          - const: nvidia,tegra124-xusb-padctl
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    items:
> +      - description: XUSB pad controller interrupt
> +
> +  resets:
> +    items:
> +      - description: pad controller reset
> +
> +  reset-names:
> +    items:
> +      - const: padctl
> +
> +  avdd-pll-utmip-supply:
> +    description: UTMI PLL power supply. Must supply 1.8 V.
> +
> +  avdd-pll-erefe-supply:
> +    description: PLLE reference PLL power supply. Must supply 1.05 V.
> +
> +  avdd-pex-pll-supply:
> +    description: PCIe/USB3 PLL power supply. Must supply 1.05 V.
> +
> +  hvdd-pex-pll-e-supply:
> +    description: High-voltage PLLE power supply. Must supply 3.3 V.
> +
> +  pads:
> +    description: A required child node named "pads" contains a list of
> +      subnodes, one for each of the pads exposed by the XUSB pad controller.
> +      Each pad may need additional resources that can be referenced in its
> +      pad node.
> +
> +      The "status" property is used to enable or disable the use of a pad.
> +      If set to "disabled", the pad will not be used on the given board. In
> +      order to use the pad and any of its lanes, this property must be set
> +      to "okay".

...or be absent.

> +    type: object
> +    properties:
> +      usb2:
> +        type: object
> +        properties:
> +          clocks:
> +            items:
> +              - description: USB2 tracking clock
> +
> +          clock-names:
> +            items:
> +              - const: trk
> +
> +          lanes:
> +            type: object
> +            properties:
> +              usb2-0:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"

Drop quotes.

> +                    enum: [ snps, xusb, uart ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"

You shouldn't need 'phandle' as the tools should handle it. If not, let 
me know and I will investigate.

> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"

Same for status. Also, a ref to dt-core.yaml is never correct.

> +
> +                additionalProperties: false

For the indented cases, I find it easier to read putting this before 
'properties'.

> +
> +              usb2-1:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ snps, xusb, uart ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              usb2-2:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ snps, xusb, uart ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +            additionalProperties: false
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +        additionalProperties: false
> +
> +      ulpi:
> +        type: object
> +        properties:
> +          lanes:
> +            type: object
> +            properties:
> +              ulpi-0:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ snps, xusb ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +            additionalProperties: false
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +        additionalProperties: false
> +
> +      hsic:
> +        type: object
> +        properties:
> +          clocks:
> +            items:
> +              - description: HSIC tracking clock
> +
> +          clock-names:
> +            items:
> +              - const: trk
> +
> +          lanes:
> +            type: object
> +            properties:
> +              hsic-0:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ snps, xusb ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              hsic-1:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ snps, xusb ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +            additionalProperties: false
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +        additionalProperties: false
> +
> +      pcie:
> +        type: object
> +        properties:
> +          clocks:
> +            items:
> +              - description: PLLE clock
> +
> +          clock-names:
> +            items:
> +              - const: pll
> +
> +          resets:
> +            items:
> +              - description: reset for the PCIe UPHY block
> +
> +          reset-names:
> +            items:
> +              - const: phy
> +
> +          lanes:
> +            type: object
> +            properties:
> +              pcie-0:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ pcie, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              pcie-1:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ pcie, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              pcie-2:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ pcie, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              pcie-3:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ pcie, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +              pcie-4:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ pcie, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +            additionalProperties: false
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +        additionalProperties: false
> +
> +      sata:
> +        type: object
> +        properties:
> +          resets:
> +            items:
> +              - description: reset for the SATA UPHY block
> +
> +          reset-names:
> +            items:
> +              - const: phy
> +
> +          lanes:
> +            type: object
> +            properties:
> +              sata-0:
> +                type: object
> +                properties:
> +                  "#phy-cells":
> +                    const: 0
> +
> +                  nvidia,function:
> +                    description: Function selection for this lane.
> +                    $ref: "/schemas/types.yaml#/definitions/string"
> +                    enum: [ sata, usb3-ss ]
> +
> +                  phandle:
> +                    $ref: "/schemas/dt-core.yaml#/properties/phandle"
> +
> +                  status:
> +                    $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +                additionalProperties: false
> +
> +            additionalProperties: false
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +        additionalProperties: false
> +
> +    additionalProperties: false
> +
> +  ports:
> +    description: A required child node named "ports" contains a list of
> +      subnodes, one for each of the ports exposed by the XUSB pad controller.
> +      Each port may need additional resources that can be referenced in its
> +      port node.
> +
> +      The "status" property is used to enable or disable the use of a port.
> +      If set to "disabled", the port will not be used on the given board. In
> +      order to use the port, this property must be set to "okay".
> +    type: object
> +    properties:
> +      usb2-0:
> +        type: object
> +        properties:
> +          # no need to further describe this because the connector will
> +          # match on gpio-usb-b-connector or usb-b-connector and cause
> +          # that binding to be selected for the subnode
> +          connector:
> +            type: object
> +
> +          mode:
> +            description: A string that determines the mode in which to
> +              run the port.
> +            $ref: "/schemas/types.yaml#/definitions/string"
> +            enum: [ host, peripheral, otg ]
> +
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          usb-role-switch:
> +            description: |
> +              A boolean property whole presence indicates that the port
> +              supports OTG or peripheral mode. If present, the port
> +              supports switching between USB host and peripheral roles.
> +              A connector must be added as a subnode in that case.
> +
> +              See ../connector/usb-connector.yaml.
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      usb2-1:
> +        type: object
> +        properties:
> +          # no need to further describe this because the connector will
> +          # match on gpio-usb-b-connector or usb-b-connector and cause
> +          # that binding to be selected for the subnode
> +          connector:
> +            type: object
> +
> +          mode:
> +            description: A string that determines the mode in which to
> +              run the port.
> +            $ref: "/schemas/types.yaml#/definitions/string"
> +            enum: [ host, peripheral, otg ]
> +
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          usb-role-switch:
> +            description: |
> +              A boolean property whole presence indicates that the port
> +              supports OTG or peripheral mode. If present, the port
> +              supports switching between USB host and peripheral roles.
> +              A connector must be added as a subnode in that case.
> +
> +              See ../connector/usb-connector.yaml.
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      usb2-2:
> +        type: object
> +        properties:
> +          # no need to further describe this because the connector will
> +          # match on gpio-usb-b-connector or usb-b-connector and cause
> +          # that binding to be selected for the subnode
> +          connector:
> +            type: object
> +
> +          mode:
> +            description: A string that determines the mode in which to
> +              run the port.
> +            $ref: "/schemas/types.yaml#/definitions/string"
> +            enum: [ host, peripheral, otg ]
> +
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          usb-role-switch:
> +            description: |
> +              A boolean property whole presence indicates that the port
> +              supports OTG or peripheral mode. If present, the port
> +              supports switching between USB host and peripheral roles.
> +              A connector must be added as a subnode in that case.
> +
> +              See ../connector/usb-connector.yaml.
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      ulpi-0:
> +        type: object
> +        properties:
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      hsic-0:
> +        type: object
> +        properties:
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      hsic-1:
> +        type: object
> +        properties:
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      usb3-0:
> +        type: object
> +        properties:
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          nvidia,usb2-companion:
> +            description: A single cell that specifies the physical port
> +              number to map this super-speed USB port to. The range of
> +              valid port numbers varies with the SoC generation.
> +            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            enum: [ 0, 1, 2 ]
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +      usb3-1:
> +        type: object
> +        properties:
> +          nvidia,internal:
> +            description: A boolean property whose presence determines
> +              that a port is internal. In the absence of this property
> +              the port is considered to be external.
> +            $ref: "/schemas/types.yaml#/definitions/flag"
> +
> +          nvidia,usb2-companion:
> +            description: A single cell that specifies the physical port
> +              number to map this super-speed USB port to. The range of
> +              valid port numbers varies with the SoC generation.
> +            $ref: "/schemas/types.yaml#/definitions/uint32"
> +            enum: [ 0, 1, 2 ]
> +
> +          status:
> +            $ref: "/schemas/dt-core.yaml#/properties/status"
> +
> +          vbus-supply:
> +            description: A phandle to the regulator supplying the VBUS
> +              voltage.
> +
> +        additionalProperties: false
> +
> +    additionalProperties: false
> +
> +additionalProperties: false
> +
> +required:
> +  - compatible
> +  - reg
> +  - resets
> +  - reset-names
> +  - avdd-pll-utmip-supply
> +  - avdd-pll-erefe-supply
> +  - avdd-pex-pll-supply
> +  - hvdd-pex-pll-e-supply
> +
> +examples:
> +  # Tegra124 and Tegra132
> +  - |
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +
> +    padctl@7009f000 {
> +        compatible = "nvidia,tegra124-xusb-padctl";
> +        reg = <0x7009f000 0x1000>;
> +        interrupts = <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>;
> +        resets = <&tegra_car 142>;
> +        reset-names = "padctl";
> +
> +        avdd-pll-utmip-supply = <&vddio_1v8>;
> +        avdd-pll-erefe-supply = <&avdd_1v05_run>;
> +        avdd-pex-pll-supply = <&vdd_1v05_run>;
> +        hvdd-pex-pll-e-supply = <&vdd_3v3_lp0>;
> +
> +        pads {
> +            usb2 {
> +                lanes {
> +                    usb2-0 {
> +                        nvidia,function = "xusb";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    usb2-1 {
> +                        nvidia,function = "xusb";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    usb2-2 {
> +                        nvidia,function = "xusb";
> +                        #phy-cells = <0>;
> +                    };
> +                };
> +            };
> +
> +            ulpi {
> +                lanes {
> +                    ulpi-0 {
> +                        status = "disabled";

The normal comment is 'status' should not be in examples, and I have a 
schema to check just that. But I guess it makes sense here with how it 
is used...

> +                        #phy-cells = <0>;
> +                    };
> +                };
> +            };
> +
> +            hsic {
> +                lanes {
> +                    hsic-0 {
> +                        status = "disabled";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    hsic-1 {
> +                        status = "disabled";
> +                        #phy-cells = <0>;
> +                    };
> +                };
> +            };
> +
> +            pcie {
> +                lanes {
> +                    pcie-0 {
> +                        nvidia,function = "usb3-ss";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    pcie-1 {
> +                        status = "disabled";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    pcie-2 {
> +                        nvidia,function = "pcie";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    pcie-3 {
> +                        status = "disabled";
> +                        #phy-cells = <0>;
> +                    };
> +
> +                    pcie-4 {
> +                        nvidia,function = "pcie";
> +                        #phy-cells = <0>;
> +                    };
> +                };
> +            };
> +
> +            sata {
> +                lanes {
> +                    sata-0 {
> +                        nvidia,function = "sata";
> +                        #phy-cells = <0>;
> +                    };
> +                };
> +            };
> +        };
> +
> +        ports {
> +            /* Micro A/B */
> +            usb2-0 {
> +                mode = "otg";
> +            };
> +
> +            /* Mini PCIe */
> +            usb2-1 {
> +                mode = "host";
> +            };
> +
> +            /* USB3 */
> +            usb2-2 {
> +                vbus-supply = <&vdd_usb3_vbus>;
> +                mode = "host";
> +            };
> +
> +            ulpi-0 {
> +                status = "disabled";
> +            };
> +
> +            hsic-0 {
> +                status = "disabled";
> +            };
> +
> +            hsic-1 {
> +                status = "disabled";
> +            };
> +
> +            usb3-0 {
> +                nvidia,usb2-companion = <2>;
> +            };
> +
> +            usb3-1 {
> +                status = "disabled";
> +            };
> +        };
> +    };

Similar comments for the rest.

Rob

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

end of thread, other threads:[~2022-11-30 17:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-18 14:40 [PATCH v2] dt-bindings: phy: tegra-xusb: Convert to json-schema Thierry Reding
2022-11-30 17:05 ` Rob Herring

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.