Skip to content

phylib: add hardware resolved pause mode support for marvell PHYs

Russell King - ARM Linux admin netdev 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 4 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 4 patches the cover letter announces are in this archive.

Reviews trailers per patch

PatchAndrew Lunn
SeriesR-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH net-next 0/4] phylib: add hardware resolved pause mode support for marvell PHYs

Hi,

Marvell PHYs provide a register that gives us the resolution results for speed, duplex and the pause enablement status. This series adds support for using this, and passing it through phylib via the newly introduced phy_get_pause() interface.

In order to do this, we need to read the copper PHY results from the correct register; now that I have access to the Marvell documentation, this becomes much easier for the 88x3310 PHY.

I also find that the code in the marvell 1G driver does not correctly check that the resolved state in the status register is valid prior to using it, so there is a patch for that too; I don't deem it important enough for it to be sent via the -net tree, which would add additional dependency complexity for this series.

 drivers/net/phy/marvell.c    |  46 +++++++++++++-
 drivers/net/phy/marvell10g.c | 147 ++++++++++++++++++++++++++++---------------
 drivers/net/phy/phy_device.c |   6 ++
 include/linux/phy.h          |   9 +++
 4 files changed, 154 insertions(+), 54 deletions(-)
Signature · 3 lines
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up