Hi Kever,
On Feb 19, 2025 at 07:02:25, Kever Yang <kever.y...@rock-chips.com> wrote:
> Hi Justin,
>
> On 2025/2/1 01:06, Justin Klaassen wrote:
>
> The patch enables the second USB3.0 Type-A USB port on the NanoPi R4S
>
> board, which prevents a crash when initializing the usb system in U-Boot
>
> and allows both Type-A USB ports to be used for booting.
>
>
> => usb start
>
> starting USB...
>
> Bus usb@fe380000: USB EHCI 1.00
>
> Bus usb@fe3c0000: "Synchronous Abort" handler, esr 0x96000010, far 0x0
>
>
> Signed-off-by: Justin Klaassen <jus...@tidylabs.net>
>
>
> ---
>
>
> arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi | 5 +++++
>
> 1 file changed, 5 insertions(+)
>
>
> diff --git a/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
> b/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
>
> index 69800cc368d..931499b084c 100644
>
> --- a/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
>
> +++ b/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
>
> @@ -36,3 +36,8 @@
>
> };
>
> };
>
> };
>
> +
>
> +&u2phy1_host {
>
> + phy-supply = <&vdd_5v>;
>
> + status = "okay";
>
> +};
>
>
> This was disabled in upstream kernel in
> dts/upstream/src/arm64/rockchip/rk3399-nanopi-r4s.dtsi
>
> Can you also update the upstream kernel dts?
>
I have sent a patch[1] to make the change upstream in the kernel dts.
[1] https://lore.kernel.org/all/20250224170925.39126-1-jus...@tidylabs.net/
> this u2phy1_host should always have the same status as usb_host1_ehci.
>
>
> Thanks,
>
> - Kever
>
>
>
Thanks,
Justin
