<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Hi Andrew,<br>
    </p>
    <div class="moz-cite-prefix">On 2021/6/10 下午8:15, Andrew Lunn wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:YMICTvjyEAgPMH9u@lunn.ch">
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">The manual does not tell how much ps a unit is.

I am confirming with Ingenic, but there is no reply

at the moment. Can we follow Rockchip's approach?

According to the description in "rockchip-dwmac.yaml"

and the related code in "dwmac-rk.c", it seems that their

delay parameter seems to be the value used by the hardware

directly instead of ps.
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">
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.</pre>
    </blockquote>
    <p><br>
    </p>
    <p><b>I did an experiment, when the tx delay is not set, RGMII works
        a <br>
      </b></p>
    <p><b>little abnormally. It seems I have to wait for Ingenic's
        reply...</b><br>
    </p>
    <p><br>
    </p>
    <blockquote type="cite" cite="mid:YMICTvjyEAgPMH9u@lunn.ch">
      <pre class="moz-quote-pre" wrap="">

    Andrew
</pre>
    </blockquote>
  </body>
</html>