Thread (29 messages) 29 messages, 4 authors, 2025-01-22

Re: 回覆: 回覆: 回覆: [PATCH v2 05/10] ARM: dts: aspeed: system1: Add RGMII support

From: Andrew Lunn <andrew@lunn.ch>
Date: 2025-01-13 14:39:14
Also in: linux-aspeed, linux-devicetree, lkml, netdev

On Mon, Jan 13, 2025 at 08:26:08AM -0600, Ninad Palsule wrote:
Hi Jacky,

On 1/13/25 00:22, Jacky Chou wrote:
quoted
Hi Ninad,
quoted
Thanks. When are you planning to push this change? I might need to hold on to
mac changes until then.
Yes. We have a plan to push the patch about RGMII delay configuration.
Currently, I try to align our SDK to kernel.
Thanks. What will be the "phy-mode" value after you make these changes?

Will it be "rgmii-id" for MAC1..4?
It should be.
If that is the case then I can test it with current configuration which may
add extra delays in the RX from PHY side.
I would then expect traffic will not flow correctly, and your see CRC
errors, because of double delays. It is however a useful test, because
if it does somehow work, it probably means the PHY driver is also
broken with its handling of delays. I don't know what PHY driver you
are using, but those in mainline should be correct, it is something i
try to review carefully.

	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