On 12/27/2023 2:33 AM, Vitaly Lifshits wrote:
On some Meteor Lake systems accessing the PHY via the MDIO interface may
result in an MDI error. This issue happens sporadically and in most cases
a second access to the PHY via the MDIO interface results in success.
As a workaround, introduce a retry counter which is set to 3 on Meteor
Lake systems. The driver will only return an error if 3 consecutive PHY
access attempts fail. The retry mechanism is disabled in specific flows,
where MDI errors are expected.
Fixes: cc23f4f0b6b9 ("e1000e: Add support for Meteor Lake")
Suggested-by: Nikolay Mushayev <[email protected]>
Co-developed-by: Nir Efrati <[email protected]>
Signed-off-by: Nir Efrati <[email protected]>
Signed-off-by: Vitaly Lifshits <[email protected]>
---
v3: rebase to dev-queue branch
v2: fix checkpatch errors
v1: initial version
---
drivers/net/ethernet/intel/e1000e/hw.h | 2 +
drivers/net/ethernet/intel/e1000e/ich8lan.c | 33 ++++
drivers/net/ethernet/intel/e1000e/phy.c | 186 ++++++++++++--------
drivers/net/ethernet/intel/e1000e/phy.h | 2 +
4 files changed, 151 insertions(+), 72 deletions(-)
This patch conflicts with your other patch [1]. In these situations
please either send the patches together as a series or wait until the
first is applied before sending the other. I've applied the other patch;
please rebase and re-send this.
Thanks,
Tony
[1]
https://lore.kernel.org/intel-wired-lan/[email protected]/
