mbox series

[net-next,v4,0/2] Use PHYlib for reset randomization and adjustable polling

Message ID 20250210082358.200751-1-o.rempel@pengutronix.de (mailing list archive)
Headers show
Series Use PHYlib for reset randomization and adjustable polling | expand

Message

Oleksij Rempel Feb. 10, 2025, 8:23 a.m. UTC
Hi all,

This patch set tackles a DP83TG720 reset lock issue and improves PHY
polling. Rather than adding a separate polling worker to randomize PHY
resets, I chose to extend the PHYlib framework - which already handles
most of the needed functionality - with adjustable polling. This
approach not only addresses the DP83TG720-specific problem (where
synchronized resets can lock the link) but also lays the groundwork for
optimizing PHY stats polling across all PHY drivers. With generic PHY
stats coming in, we can adjust the polling interval based on hardware
characteristics, such as using longer intervals for PHYs with stable HW
counters or shorter ones for high-speed links prone to counter
overflows.

Patch version changes are tracked in separate patches.

Oleksij Rempel (2):
  net: phy: Add support for driver-specific next update time
  net: phy: dp83tg720: Add randomized polling intervals for link
    detection

 drivers/net/phy/dp83tg720.c | 78 +++++++++++++++++++++++++++++++++++++
 drivers/net/phy/phy.c       | 21 +++++++++-
 include/linux/phy.h         | 13 +++++++
 3 files changed, 111 insertions(+), 1 deletion(-)

--
2.39.5

Comments

patchwork-bot+netdevbpf@kernel.org Feb. 12, 2025, 7:20 p.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 10 Feb 2025 09:23:56 +0100 you wrote:
> Hi all,
> 
> This patch set tackles a DP83TG720 reset lock issue and improves PHY
> polling. Rather than adding a separate polling worker to randomize PHY
> resets, I chose to extend the PHYlib framework - which already handles
> most of the needed functionality - with adjustable polling. This
> approach not only addresses the DP83TG720-specific problem (where
> synchronized resets can lock the link) but also lays the groundwork for
> optimizing PHY stats polling across all PHY drivers. With generic PHY
> stats coming in, we can adjust the polling interval based on hardware
> characteristics, such as using longer intervals for PHYs with stable HW
> counters or shorter ones for high-speed links prone to counter
> overflows.
> 
> [...]

Here is the summary with links:
  - [net-next,v4,1/2] net: phy: Add support for driver-specific next update time
    https://git.kernel.org/netdev/net-next/c/8bf47e4d7b87
  - [net-next,v4,2/2] net: phy: dp83tg720: Add randomized polling intervals for link detection
    https://git.kernel.org/netdev/net-next/c/e252af1a67fe

You are awesome, thank you!