Re: [PATCH v2 2/2] arm64: dts: rockchip: rk3399: fix pd_tcpc0 and pd_tcpc1 node position
Hi Heiko, 在 2020/5/19 上午6:29, Heiko Stübner 写道: Hi Kever, Caesar, could you double check where the type-c power-domains are located in the power-tree, as Caesar did add them under pd_vio back in 2016. Johan's patch is correct, the pd_tcpc0 and pd_tcpc1 are grouped by VDD_LOGIC. I have a passed test for pd_vio without pd_tcpc*. Thanks Heiko Am Dienstag, 28. April 2020, 22:30:03 CEST schrieb Johan Jonker: The pd_tcpc0 and pd_tcpc1 nodes are currently a sub node of pd_vio. In the rk3399 TRM figure of the 'Power Domain Partition' and in the table of 'Power Domain and Voltage Domain Summary' these power domains are positioned directly under VD_LOGIC, so fix that in 'rk3399.dtsi'. Signed-off-by: Johan Jonker Reviewed-by: Caesar Wang Thanks, -Caesar --- arch/arm64/boot/dts/rockchip/rk3399.dtsi | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi index 37279db53..a4dc1bf2e 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi @@ -1056,6 +1056,16 @@ clocks = < HCLK_SDIO>; pm_qos = <_sdioaudio>; }; + pd_tcpc0@RK3399_PD_TCPD0 { + reg = ; + clocks = < SCLK_UPHY0_TCPDCORE>, +< SCLK_UPHY0_TCPDPHY_REF>; + }; + pd_tcpc1@RK3399_PD_TCPD1 { + reg = ; + clocks = < SCLK_UPHY1_TCPDCORE>, +< SCLK_UPHY1_TCPDPHY_REF>; + }; pd_usb3@RK3399_PD_USB3 { reg = ; clocks = < ACLK_USB3>; @@ -1088,16 +1098,6 @@ pm_qos = <_isp1_m0>, <_isp1_m1>; }; - pd_tcpc0@RK3399_PD_TCPD0 { - reg = ; - clocks = < SCLK_UPHY0_TCPDCORE>, -< SCLK_UPHY0_TCPDPHY_REF>; - }; - pd_tcpc1@RK3399_PD_TCPD1 { - reg = ; - clocks = < SCLK_UPHY1_TCPDCORE>, -< SCLK_UPHY1_TCPDPHY_REF>; - }; pd_vo@RK3399_PD_VO { reg = ; #address-cells = <1>;
Re: [PATCH v2 2/2] arm64: dts: rockchip: rk3399: fix pd_tcpc0 and pd_tcpc1 node position
Hi Kever, Caesar, could you double check where the type-c power-domains are located in the power-tree, as Caesar did add them under pd_vio back in 2016. Thanks Heiko Am Dienstag, 28. April 2020, 22:30:03 CEST schrieb Johan Jonker: > The pd_tcpc0 and pd_tcpc1 nodes are currently a sub node of pd_vio. > In the rk3399 TRM figure of the 'Power Domain Partition' and in the > table of 'Power Domain and Voltage Domain Summary' these power domains > are positioned directly under VD_LOGIC, so fix that in 'rk3399.dtsi'. > > Signed-off-by: Johan Jonker > --- > arch/arm64/boot/dts/rockchip/rk3399.dtsi | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi > b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > index 37279db53..a4dc1bf2e 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > @@ -1056,6 +1056,16 @@ > clocks = < HCLK_SDIO>; > pm_qos = <_sdioaudio>; > }; > + pd_tcpc0@RK3399_PD_TCPD0 { > + reg = ; > + clocks = < SCLK_UPHY0_TCPDCORE>, > + < SCLK_UPHY0_TCPDPHY_REF>; > + }; > + pd_tcpc1@RK3399_PD_TCPD1 { > + reg = ; > + clocks = < SCLK_UPHY1_TCPDCORE>, > + < SCLK_UPHY1_TCPDPHY_REF>; > + }; > pd_usb3@RK3399_PD_USB3 { > reg = ; > clocks = < ACLK_USB3>; > @@ -1088,16 +1098,6 @@ > pm_qos = <_isp1_m0>, ><_isp1_m1>; > }; > - pd_tcpc0@RK3399_PD_TCPD0 { > - reg = ; > - clocks = < SCLK_UPHY0_TCPDCORE>, > - < SCLK_UPHY0_TCPDPHY_REF>; > - }; > - pd_tcpc1@RK3399_PD_TCPD1 { > - reg = ; > - clocks = < SCLK_UPHY1_TCPDCORE>, > - < SCLK_UPHY1_TCPDPHY_REF>; > - }; > pd_vo@RK3399_PD_VO { > reg = ; > #address-cells = <1>; >
[PATCH v2 2/2] arm64: dts: rockchip: rk3399: fix pd_tcpc0 and pd_tcpc1 node position
The pd_tcpc0 and pd_tcpc1 nodes are currently a sub node of pd_vio. In the rk3399 TRM figure of the 'Power Domain Partition' and in the table of 'Power Domain and Voltage Domain Summary' these power domains are positioned directly under VD_LOGIC, so fix that in 'rk3399.dtsi'. Signed-off-by: Johan Jonker --- arch/arm64/boot/dts/rockchip/rk3399.dtsi | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi index 37279db53..a4dc1bf2e 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi @@ -1056,6 +1056,16 @@ clocks = < HCLK_SDIO>; pm_qos = <_sdioaudio>; }; + pd_tcpc0@RK3399_PD_TCPD0 { + reg = ; + clocks = < SCLK_UPHY0_TCPDCORE>, +< SCLK_UPHY0_TCPDPHY_REF>; + }; + pd_tcpc1@RK3399_PD_TCPD1 { + reg = ; + clocks = < SCLK_UPHY1_TCPDCORE>, +< SCLK_UPHY1_TCPDPHY_REF>; + }; pd_usb3@RK3399_PD_USB3 { reg = ; clocks = < ACLK_USB3>; @@ -1088,16 +1098,6 @@ pm_qos = <_isp1_m0>, <_isp1_m1>; }; - pd_tcpc0@RK3399_PD_TCPD0 { - reg = ; - clocks = < SCLK_UPHY0_TCPDCORE>, -< SCLK_UPHY0_TCPDPHY_REF>; - }; - pd_tcpc1@RK3399_PD_TCPD1 { - reg = ; - clocks = < SCLK_UPHY1_TCPDCORE>, -< SCLK_UPHY1_TCPDPHY_REF>; - }; pd_vo@RK3399_PD_VO { reg = ; #address-cells = <1>; -- 2.11.0