Thread (2 messages) 2 messages, 2 authors, 3d ago

Re: [PATCH net-next] net: phy: aquantia: Poll global status register for provision ID

flat view

From: Andrew Lunn <andrew@lunn.ch>
Date: 2026-10-29 13:35:14
Also in: lkml

On Thu, Oct 29, 2026 at 05:28:10PM +0800, Hongmei Xu wrote:
After the firmware reset sequence completes, the global reserved status
register (VEND1_GLOBAL_RSVD_STAT1) may not yet contain a valid provision
ID. This can cause subsequent firmware operations to proceed with stale
state.

Poll VEND1_GLOBAL_RSVD_STAT1 until the provision ID field is non-zero
before returning from aqr_wait_reset_complete(). A timeout is treated as
a non-fatal warning since the PHY can still function without a valid
provision ID.
If it can still work without the provisioning ID, why bother reading
it?

Maybe extend the commit message saying what features will be missing
if there is no ID.

It looks like USXGMII and 10G-QXGMII might be mixed up? Is such a mix
up not fatal?

	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