Thread (2 messages) 2 messages, 2 authors, 2021-06-13

Re: [PATCH v2 2/2] net: stmmac: Add Ingenic SoCs MAC support.

From: 周琰杰 <hidden>
Date: 2021-06-13 08:26:39
Also in: linux-arm-kernel, linux-devicetree, linux-mips, lkml

Possibly related (same subject, not in this thread)

Hi Andrew,

于 Thu, 10 Jun 2021 16:42:24 +0200
Andrew Lunn [off-list ref] 写道:
quoted
    We are much more strict about this now than before. You have to
use standard units and convert to hardware values. It also makes it
a lot easier for DT writers, if they have an idea what the units
mean.

    Having the MAC add small delays is something you can add later,
    without breaking backwards compatibility. So if you cannot
determine what the units are now, just submit the glue driver
without support for this feature. If anybody really needs it, they
can do the needed research, maybe do some measurements, and then
add the code.


I did an experiment, when the tx delay is not set, RGMII works a  
You had rgmii-id in your device tree, so that the PHY added the
delays?
I have tried rgmii-id and rgmii-txid. If we don’t add a fine-tuning
parameter, it still can’t work properly. In these two modes, we still
need to add about 500ps delay on the mac side to ensure it works
properly.

Thanks and best regards!
	Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help