Oh okay, I was not aware of this issue and have not encountered it yet. But wouldn't it be more appropriate to get u-boot to power cycle the host controller at boot? From: Soeren Moch <smoch@xxxxxx> Sent: Tuesday, April 27, 2021 5:07 PM To: Christian Löhle <CLoehle@xxxxxxxxxxxxxx>; robh+dt@xxxxxxxxxx <robh+dt@xxxxxxxxxx>; heiko@xxxxxxxxx <heiko@xxxxxxxxx>; jbx6244@xxxxxxxxx <jbx6244@xxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx <devicetree@xxxxxxxxxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; linux-rockchip@xxxxxxxxxxxxxxxxxxx <linux-rockchip@xxxxxxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx> Subject: Re: [PATCH] arm64: dts: rockchip: include uhs support for rockpro64 On 27.04.21 08:20, Christian L?hle wrote: > The DesignWare Host Controller has full UHS-I support, so use it. Enabling this UHS support makes 'reboot' hang when booting the RockPro64 from SD card. It would work when booting from eMMC, or with a modified ATF which does a power cycle on reboot. But for general use it is not save to enable UHS support, or did I miss some recent changes? Regards, Soeren > > Signed-off-by: Christian Loehle <cloehle@xxxxxxxxxxxxxx> > --- > arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi > index 6bff8db7d33e..d22a489ec214 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi > @@ -722,6 +722,7 @@ &sdmmc { > max-frequency = <150000000>; > pinctrl-names = "default"; > pinctrl-0 = <&sdmmc_clk &sdmmc_cmd &sdmmc_bus4>; > + sd-uhs-sdr104; > vmmc-supply = <&vcc3v0_sd>; > vqmmc-supply = <&vcc_sdio>; > status = "okay"; Hyperstone GmbH | Line-Eid-Strasse 3 | 78467 Konstanz Managing Directors: Dr. Jan Peter Berns. Commercial register of local courts: Freiburg HRB381782