Re: [PATCH 6/6] arm64: dts: allwinner: a523: x96qpro+: enable Ethernet support

On Tue, Aug 4, 2026 at 10:53 AM Andrew Lunn [off-list ref] wrote:

quoted

+&gmac1 {
+     clocks = <&ccu CLK_BUS_EMAC1>, <&ccu CLK_MBUS_EMAC1>;
+     pinctrl-names = "default";
+     pinctrl-0 = <&rgmii1_pins>;
+     phy-mode = "rgmii-id";
+     phy-handle = <&ext_rgmii_phy>;
+     phy-supply = <&reg_cldo3>;
+
+     tx-internal-delay-ps = <200>;
+     rx-internal-delay-ps = <500>;

500ns is quite big. How did you come to this value? Do you have the
schematic & gerber files? Does the lines look unbalanced?

Unit is ps, not ns.


ChenYu