Re: [PATCH net-next 0/9] net: dsa: vsc73xx: fix MDIO bus access and PHY operations

From: Andrew Lunn
Date: Mon Jul 29 2024 - 18:54:26 EST


On Mon, Jul 29, 2024 at 11:06:06PM +0200, Pawel Dembicki wrote:
> The VSC73xx driver has issues with PHY configuration. This patch series
> fixes most of them.
>
> The first patch fixes the phylink capabilities, because the MAC in the
> vsc73xx family doesn't handle 1000BASE HD mode.
>
> The second patch synchronizes the register configuration routine with the
> datasheet recommendations.
>
> Patches 3-5 restore proper communication on the MDIO bus. Currently,
> the write value isn't sent to the MDIO register, and without a mutex,
> communication with the PHY can be interrupted. This causes the PHY to
> receive improper configuration and autonegotiation could fail.
>
> The sixth patch speeds up the internal MDIO bus to the maximum value
> allowed by the datasheet.
>
> The seventh patch removes the PHY reset blockade, as it is no longer
> required.
>
> After fixing the MDIO operations, autonegotiation became possible.
> The eighth patch removes the blockade, which became unnecessary after
> the MDIO operations fix. It also enables the MDI-X feature, which is
> disabled by default in forced 100BASE-TX mode like other Vitesse PHYs.
>
> The last patch implements the downshift feature and enables it by default.

Please separate fixes from new development. Fixed should target net,
while new features should be for net-next.

Andrew