On 3/3/2024 12:51, Vitaly Lifshits wrote:
Forcing SMBUS inside the ULP enabling flow leads to sporadic PHY loss on some systems. It is suspected to be caused by initiating PHY transactions before the interface settles.Separating this configuration from the ULP enabling flow and moving it to the shutdown function allows enough time for the interface to settle and avoids adding a delay. Fixes: 6607c99e7034 ("e1000e: i219 - fix to enable both ULP and EEE in Sx state") Signed-off-by: Vitaly Lifshits <[email protected]> Co-developed-by: Dima Ruinskiy <[email protected]> Signed-off-by: Dima Ruinskiy <[email protected]> --- v2: address community review v1: initial version --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 19 ------------------- drivers/net/ethernet/intel/e1000e/netdev.c | 18 ++++++++++++++++++ 2 files changed, 18 insertions(+), 19 deletions(-)
Tested-by: Naama Meir <[email protected]>
