Re: [PATCH 4/5] arm64: dts: allwinner: a527: add EMAC0 to Radxa A5E board

Andrew Lunn
2025-04-23T16:59:12Z

> +&emac0 {
> + phy-mode = "rgmii";

Does the PCB have extra long clock lines in order to provide the
needed 2ns delay? I guess not, so this should be rgmii-id.

> + phy-handle = <&ext_rgmii_phy>;
> +
> + allwinner,tx-delay-ps = <300>;
> + allwinner,rx-delay-ps = <400>;

These are rather low delays, since the standard requires 2ns. Anyway,
once you change phy-mode, you probably don't need these.

Andrew