Re: [PATCH 1/3] dt-bindings: net: nuvoton: Add schema for Nuvoton MA35 family GMAC

From: Joey Lu
Date: Thu Nov 07 2024 - 05:31:42 EST


Dear Andrew,

Thank you for your reply.

Andrew Lunn 於 11/7/2024 2:13 AM 寫道:
+ phy-mode:
+ enum:
+ - rmii
+ - rgmii-id
The phy-mode deepened on the board design. All four rgmii values are
valid.
I will add them.
+
+ tx_delay:
+ maxItems: 1
+ description:
+ Control transmit clock path delay in nanoseconds.
+
+ rx_delay:
+ maxItems: 1
+ description:
+ Control receive clock path delay in nanoseconds.
If you absolutely really need these, keep them, but i suggest you drop
them. They just cause confusion, when ideally we want the PHY to be
adding RGMII delays, not the MAC.

If you do need them, then they should be in pS.

I will fix it.

We have customers who use a fixed link instead of a PHY, so these properties may be necessary.

Andrew

Thanks!

BR,

Joey