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

From: Andrew Lunn

Date: Thu Oct 29 2026 - 09:36:43 EST


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