Re: [PATCH] arm64: dts: rockchip: support the usb2phy for rk3399 evb

From: Heiko Stuebner
Date: Thu Jul 07 2016 - 06:11:57 EST


Hi,

Am Donnerstag, 7. Juli 2016, 12:46:40 schrieb Caesar Wang:
> From: Frank Wang <frank.wang@xxxxxxxxxxxxxx>
>
> 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 <frank.wang@xxxxxxxxxxxxxx>
> Signed-off-by: Caesar Wang <wxt@xxxxxxxxxxxxxx>
> ---
>
> 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.


Heiko