[PATCH v3 net 0/3] amd-xgbe: RX adaptation and PHY handling fixes

Raju Rangoju posted 3 patches 1 month ago
drivers/net/ethernet/amd/xgbe/xgbe-drv.c    | 19 ++---
drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c | 82 +++++++++++++++++++--
drivers/net/ethernet/amd/xgbe/xgbe.h        |  4 +
3 files changed, 89 insertions(+), 16 deletions(-)
[PATCH v3 net 0/3] amd-xgbe: RX adaptation and PHY handling fixes
Posted by Raju Rangoju 1 month ago
This series fixes several issues in the amd-xgbe driver related to RX
adaptation and PHY handling in 10GBASE-KR mode, particularly when
auto-negotiation is disabled.

Patch 1 fixes link status handling during RX adaptation by correctly
reading the latched link status bit so transient link drops are
detected without losing the current state.

Patch 2 prevents CRC errors that can occur when performing RX
adaptation with auto-negotiation turned off. The driver now stops
TX/RX before re-triggering RX adaptation and only re-enables traffic
once adaptation completes and the link is confirmed up, ensuring
packets are not corrupted during the adaptation window.

Patch 3 restores the intended ordering of PHY reset relative to
phy_start(), making sure PHY settings are reset before the PHY is
started instead of afterwards.

Changes since v1:
 - change the data_path_stopped flag to boolean type
   as it is only used as a true/false indicator.

Changes since v2:
 - use the correct fixes tag for the link status handling patch
 - change the data_path_stopped flag to be cleared in phy_start() to
   ensure it is reset on device restart

Raju Rangoju (3):
  amd-xgbe: fix link status handling in xgbe_rx_adaptation
  amd-xgbe: prevent CRC errors during RX adaptation with AN disabled
  amd-xgbe: reset PHY settings before starting PHY

 drivers/net/ethernet/amd/xgbe/xgbe-drv.c    | 19 ++---
 drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c | 82 +++++++++++++++++++--
 drivers/net/ethernet/amd/xgbe/xgbe.h        |  4 +
 3 files changed, 89 insertions(+), 16 deletions(-)

-- 
2.34.1