[PATCH net-next v3 01/10] net: stmmac: move XPCS lifetime management to platform drivers

Coia Prant coiaprant at gmail.com
Thu Sep 3 01:51:31 PDT 2026


Maxime Chevallier <maxime.chevallier at bootlin.com> 于2026年9月3日周四 16:32写道:
>
> Hi again,
>
> Sorry I had a second look at that after a good night's sleep, and actually here
> we may regress. The original logic checked for "pcs-handle" presence, then
> for the pcs mask, but if no PCS is found we didn't error out, we returned 0.
>
> This is making it mandatory to have a PCS. Please handle gracefully the "there's no
> PCS" case :(
>
>
> Maxime

Hi,

This change covers all callbacks that use `select_pcs`. Drivers
without `select_pcs` callbacks will not have their pcs used by
Phylink.

Currently, only the Intel mgbe driver uses xpcs, and Intel has also
set a mask for pcs-handle not set.

This means that on Intel platforms, the parts that implement
`select_pcs` always require xpcs to exist.

I think there shouldn't be any problems here, but I agree with you,
regression issues can indeed easily occur here.

Best,
Coia



More information about the linux-arm-kernel mailing list