Re: [PATCH v2 2/2] arm64: dts: rockchip: Add Radxa ROCK 4D device tree
From: Jonas Karlman
Date: Thu Feb 13 2025 - 11:11:58 EST
Hi Detlev,
On 2025-02-13 15:57, Detlev Casanova wrote:
> From: Stephen Chen <stephen@xxxxxxxxx>
>
> The Radxa ROCK 4D board is based on the Rockchip rk3576 SoC.
>
> The device tree adds support for basic devices:
> - UART
> - SD Card
> - Ethernet
> - USB
> - RTC
>
> It has 4 USB ports but only 3 are usable as the top left one is used
> for maskrom.
>
> It has a USB-C port that is only used for powering the board.
>
> Signed-off-by: Stephen Chen <stephen@xxxxxxxxx>
> Signed-off-by: Detlev Casanova <detlev.casanova@xxxxxxxxxxxxx>
> ---
> arch/arm64/boot/dts/rockchip/Makefile | 1 +
> .../boot/dts/rockchip/rk3576-rock-4d.dts | 651 ++++++++++++++++++
> 2 files changed, 652 insertions(+)
> create mode 100644 arch/arm64/boot/dts/rockchip/rk3576-rock-4d.dts
>
> diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile
> index def1222c1907e..a112aeb37948a 100644
> --- a/arch/arm64/boot/dts/rockchip/Makefile
> +++ b/arch/arm64/boot/dts/rockchip/Makefile
> @@ -132,6 +132,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-wolfvision-pf5-display-vz.dtbo
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-wolfvision-pf5-io-expander.dtbo
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3576-armsom-sige5.dtb
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3576-evb1-v10.dtb
> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3576-rock-4d.dtb
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3582-radxa-e52c.dtb
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-armsom-sige7.dtb
> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-armsom-w3.dtb
> diff --git a/arch/arm64/boot/dts/rockchip/rk3576-rock-4d.dts b/arch/arm64/boot/dts/rockchip/rk3576-rock-4d.dts
> new file mode 100644
> index 0000000000000..f356742f9d643
> --- /dev/null
> +++ b/arch/arm64/boot/dts/rockchip/rk3576-rock-4d.dts
> @@ -0,0 +1,651 @@
[snip]
> +&gmac0 {
> + phy-mode = "rgmii-id";
> + clock_in_out = "output";
> +
> + snps,reset-gpio = <&gpio2 RK_PB5 GPIO_ACTIVE_LOW>;
> + snps,reset-active-low;
> + snps,reset-delays-us = <0 20000 100000>;
The snps,reset- props are deprecated and should be changed to reset-
props in the phy node.
> +
> + pinctrl-names = "default";
> + pinctrl-0 = <ð0m0_miim
> + ð0m0_tx_bus2
> + ð0m0_rx_bus2
> + ð0m0_rgmii_clk
> + ð0m0_rgmii_bus
> + ðm0_clk0_25m_out>;
> +
> + phy-handle = <&rgmii_phy0>;
> + status = "okay";
> +};
[snip]
> +&mdio0 {
> + rgmii_phy0: phy@1 {
Maybe ethernet-phy@1 ?
> + compatible = "ethernet-phy-ieee802.3-c22";
> + reg = <0x1>;
> + clocks = <&cru REFCLKO25M_GMAC0_OUT>;
Please add reset- props here.
Changing to use reset- props may cause issue if a RTL8211F PHY is used
on the board. Use a ethernet-phy-id compatible or mainline U-Boot to
ensure the Ethernet PHY can be discovered during probe.
Regards,
Jonas
> + };
> +};
> +
[snip]