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

Chen-Yu Tsai <[email protected]> Tue, 4 Aug 2026 11:11:04 +0800

Newsgroups	gmane.linux.network,gmane.linux.drivers.devicetree,gmane.linux.ports.arm.kernel
Message-ID	<CAGb2v64yH+grXMbxg1te6XHt1PxhUc4Jzch_OdUW50tq2Eg1cw@mail.gmail.com>

On Tue, Aug 4, 2026 at 10:53 AM Andrew Lunn <[email protected]> wrote:
>
> > +&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