Thread (34 messages) flat view 34 messages, 5 authors, 12d ago

Re: [PATCH v2 06/10] net: pcs: xpcs: add ANRESTART support for SGMII link recovery

From: Andrew Lunn <andrew@lunn.ch>
Date: 2026-08-02 14:31:32
Also in: linux-arm-kernel, linux-devicetree, linux-phy, linux-renesas-soc, linux-rockchip, lkml

On Sun, Aug 02, 2026 at 11:11:40AM +0800, Coia Prant wrote:
Hi Andrew,

Thanks for the careful review.
quoted
This probably needs more wide spread testing. I would be afraid that
if you kick off an AN cycle, you loose the link until it completes?
You would of noticed that on your hardware, but other hardware
obviously behaves differently...
The trigger condition is: link is down AND an_complete is set.
Ah, i missed the "link is down" check. That makes it safer.

I still find it interesting that this devices is different. Has the
silicon vendor done the integration of the licensed IP wrong?
Regarding testing:
- Jiawen Wu has tested this on Wangxun NICs and reported no regression
  (Tested-by tag included in the patch).
Probably Maxime will test on the platform he has, but i don't know if
any use the XPCS.

	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