[lkml]   [2019]   [Apr]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v3 net-next 20/24] net: dsa: sja1105: Error out if RGMII delays are requested in DT
On Sat, Apr 13, 2019 at 04:28:18AM +0300, Vladimir Oltean wrote:
> Documentation/devicetree/bindings/net/ethernet.txt is confusing because
> it says what the MAC should not do, but not what it *should* do:
> * "rgmii-rxid" (RGMII with internal RX delay provided by the PHY, the MAC
> should not add an RX delay in this case)
> The gap in semantics is threefold:
> 1. Is it illegal for the MAC to apply the Rx internal delay by itself,
> and simplify the phy_mode (mask off "rgmii-rxid" into "rgmii") before
> passing it to of_phy_connect? The documentation would suggest yes.
> 1. For "rgmii-rxid", while the situation with the Rx clock skew is more
> or less clear (needs to be added by the PHY), what should the MAC
> driver do about the Tx delays? Is it an implicit wild card for the
> MAC to apply delays in the Tx direction if it can? What if those were
> already added as serpentine PCB traces, how could that be made more
> obvious through DT bindings so that the MAC doesn't attempt to add
> them twice and again potentially break the link?
> 3. If the interface is a fixed-link and therefore the PHY object is
> fixed (a purely software entity that obviously cannot add clock
> skew), what is the meaning of the above property?
> So an interpretation of the RGMII bindings was chosen that hopefully
> does not contradict their intention but also makes them more applied.
> The SJA1105 driver understands to act upon "rgmii-*id" phy-mode bindings
> if the port is in the PHY role (either explicitly, or if it is a
> fixed-link). Otherwise it always passes the duty of setting up delays to
> the PHY driver.

That is a good interpretation. I always recommend the PHY does the
delay, because in general the PHY can, and often the MAC cannot.

> Signed-off-by: Vladimir Oltean <>
> Reviewed-by: Florian Fainelli <>

Reviewed-by: Andrew Lunn <>


 \ /
  Last update: 2019-04-13 18:52    [W:0.232 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site