[net,v3,0/5] net: dsa: vsc73xx: fix MDIO bus access and PHY operations

Message

Pawel Dembicki Aug. 9, 2024, 7:38 p.m. UTC

This series are extracted patches from net-next series [0].

The VSC73xx driver has issues with PHY configuration. This patch series
fixes most of them.

The first patch synchronizes the register configuration routine with the
datasheet recommendations.

Patches 2-3 restore proper communication on the MDIO bus. Currently,
the write value isn't sent to the MDIO register, and without a busy check,
communication with the PHY can be interrupted. This causes the PHY to
receive improper configuration and autonegotiation could fail.

The fourth patch removes the PHY reset blockade, as it is no longer
required.

After fixing the MDIO operations, autonegotiation became possible.
The last patch removes the blockade, which became unnecessary after
the MDIO operations fix.

[0] https://patchwork.kernel.org/project/netdevbpf/list/?series=874739&state=%2A&archive=both

Pawel Dembicki (5):
  net: dsa: vsc73xx: fix port MAC configuration in full duplex mode
  net: dsa: vsc73xx: pass value in phy_write operation
  net: dsa: vsc73xx: check busy flag in MDIO operations
  net: dsa: vsc73xx: allow phy resetting
  net: phy: vitesse: repair vsc73xx autonegotiation

 drivers/net/dsa/vitesse-vsc73xx-core.c | 54 +++++++++++++++++++-------
 drivers/net/phy/vitesse.c              | 14 -------
 2 files changed, 41 insertions(+), 27 deletions(-)

Comments

[e-mail] Aug. 12, 2024, 10:50 a.m. UTC | #1

Hello:

This series was applied to netdev/net.git (main)
by David S. Miller <[e-mail]>:

On Fri,  9 Aug 2024 21:38:01 +0200 you wrote:
> This series are extracted patches from net-next series [0].
> 
> The VSC73xx driver has issues with PHY configuration. This patch series
> fixes most of them.
> 
> The first patch synchronizes the register configuration routine with the
> datasheet recommendations.
> 
> [...]

Here is the summary with links:
  - [net,v3,1/5] net: dsa: vsc73xx: fix port MAC configuration in full duplex mode
    https://git.kernel.org/netdev/net/c/63796bc2e97c
  - [net,v3,2/5] net: dsa: vsc73xx: pass value in phy_write operation
    https://git.kernel.org/netdev/net/c/5b9eebc2c7a5
  - [net,v3,3/5] net: dsa: vsc73xx: check busy flag in MDIO operations
    https://git.kernel.org/netdev/net/c/fa63c6434b6f
  - [net,v3,4/5] net: dsa: vsc73xx: allow phy resetting
    https://git.kernel.org/netdev/net/c/9f9a72654622
  - [net,v3,5/5] net: phy: vitesse: repair vsc73xx autonegotiation
    https://git.kernel.org/netdev/net/c/de7a670f8def

You are awesome, thank you!