On 08 Jun 2017, at 09:35, Meng Dongyang daniel.meng@rock-chips.com wrote:

Add fixed regulator for the port of typec0 and typec1 to control vbus instead of gpio.

Signed-off-by: Meng Dongyang daniel.meng@rock-chips.com Reviewed-by: Simon Glass sjg@chromium.org

---

Changes in v4: None Changes in v3: None

Changes in v2:

• New change, add fixed regulator for rk3399

arch/arm/dts/rk3399-evb.dts | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/rk3399-evb.dts b/arch/arm/dts/rk3399-evb.dts index f5af75b..bff00c3 100644 --- a/arch/arm/dts/rk3399-evb.dts +++ b/arch/arm/dts/rk3399-evb.dts @@ -60,6 +60,18 @@ gpio = <&gpio4 25 GPIO_ACTIVE_HIGH>; };

• vcc5v0_typec0: vcc5v0-typec0-en {

• compatible = "regulator-fixed";
  

• regulator-name = "vcc5v0_typec0";
  

• gpio = <&gpio1 3 GPIO_ACTIVE_HIGH>;
  

• };
• vcc5v0_typec1: vcc5v0-typec1-en {

• compatible = "regulator-fixed";
  

• regulator-name = "vcc5v0_typec1";
  

• gpio = <&gpio1 4 GPIO_ACTIVE_HIGH>;
  

• };

Shouldn’t these written as RK_PA3 and RK_PA4 to remain consistent with the Linux DTS?

clkin_gmac: external-gmac-clock { compatible = "fixed-clock"; clock-frequency = <125000000>; @@ -163,7 +175,7 @@ };

&dwc3_typec0 {

• rockchip,vbus-gpio = <&gpio1 3 GPIO_ACTIVE_HIGH>;

• vbus-supply = <&vcc5v0_typec0>; status = "okay";

};

@@ -176,7 +188,7 @@ };

&dwc3_typec1 {

• rockchip,vbus-gpio = <&gpio1 4 GPIO_ACTIVE_HIGH>;

• vbus-supply = <&vcc5v0_typec1>; status = "okay";

};