Hi Heiko,

On 2016/7/7 18:11, Heiko Stuebner wrote:

Hi,

Am Donnerstag, 7. Juli 2016, 12:46:40 schrieb Caesar Wang:

quoted

From: Frank Wang <redacted>

This patch adds the usb2phy needed dts node information for rk3399.

USB2.0 PHY is comprised of one Host port and one OTG port.
Host Port is for USB2.0 host controller; OTG port is for USB2.0 part of
USB3.0 OTG controller, and as a part to construct a fully feature TypeC
subsystem.

The USB2.0 vbus gpio is board specific, it's no same with all rk3399
boards, so move it into evb voard.

Signed-off-by: Frank Wang <redacted>
Signed-off-by: Caesar Wang <redacted>
---

  arch/arm64/boot/dts/rockchip/rk3399-evb.dts |  4 ++++
  arch/arm64/boot/dts/rockchip/rk3399.dtsi    | 19 +++++++++++++++++++
  2 files changed, 23 insertions(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi

b/arch/arm64/boot/dts/rockchip/rk3399.dtsi index 4c84229..21d147f 100644

--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi

@@ -242,6 +242,25 @@
  		status = "disabled";
  	};

+	usb2phy: usb2phy {
+		compatible = "rockchip,rk3399-usb-phy";
+		rockchip,grf = <&grf>;
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		usb2phy0: usb2-phy0 {
+			#phy-cells = <0>;
+			#clock-cells = <0>;
+			reg = <0xe458>;
+		};
+
+		usb2phy1: usb2-phy1 {
+			#phy-cells = <0>;
+			#clock-cells = <0>;
+			reg = <0xe468>;
+		};
+	};
+

The rk3399 uses the innosilicon usb2 phy and the binding we've come up for
it looks a lot different than the above. So I'm not sure where this is
coming from, especially as I haven't seen the driver part for the rk3399 at
all yet. So far Frank was always working on te rk3366 variant, which seemed
to be the most complete.

Yeah, rk3399 is ready for using the new driver and the binding which I 
have handed out earlier, but it seems that they have not applied yet, so 
shall we send out the new patches for rk3399 until they will be applied?

BR.
Frank

Heiko