[Intel-wired-lan] [iwl-net][PATCH v2] Revert "e1000e: move force SMBUS near the end of enable_ulp function"

2024-06-11 Thread Hui Wang
.org/show_bug.cgi?id=218940 Reported-by: Dieter Mummenschanz Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218936 Acked-by: Paul Menzel Signed-off-by: Hui Wang --- in V2: change "regression the commit ... tried to fix" to "regression introduced by commit" drivers/

Re: [Intel-wired-lan] [iwl-net][PATCH] Revert "e1000e: move force SMBUS near the end of enable_ulp function"

2024-06-10 Thread Hui Wang
On 6/10/24 22:14, Zhang, Rui wrote: On Mon, 2024-06-10 at 08:36 +0200, Paul Menzel wrote: Dear Hui, Naama also added Tested-by lines two both commits in question. Could Intel’s test coverage please extended to the problem at hand? Acked-by: Paul Menzel Plus that, 1. Todd and I can test

Re: [Intel-wired-lan] [iwl-net][PATCH] Revert "e1000e: move force SMBUS near the end of enable_ulp function"

2024-06-10 Thread Hui Wang
On 6/10/24 14:36, Paul Menzel wrote: Dear Hui, Thank you for your patch. Am 10.06.24 um 03:32 schrieb Hui Wang: This reverts commit bfd546a552e140b0a4c8a21527c39d6d21addb28 Commit bfd546a552e1 ("e1000e: move force SMBUS near the end of enable_ulp function") introduces syst

[Intel-wired-lan] [iwl-net][PATCH] Revert "e1000e: move force SMBUS near the end of enable_ulp function"

2024-06-09 Thread Hui Wang
i?id=218940 Reported-by: Dieter Mummenschanz Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218936 Signed-off-by: Hui Wang --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 22 - drivers/net/ethernet/intel/e1000e/netdev.c | 18 + 2 files changed,

[Intel-wired-lan] [iwl-net][PATCH v3] e1000e: move force SMBUS near the end of enable_ulp function

2024-05-17 Thread Hui Wang
rom enable ulp function to avoid PHY loss issue") Signed-off-by: Hui Wang Acked-by: Vitaly Lifshits Tested-by: Naama Meir Reviewed-by: Simon Horman Reviewed-by: Paul Menzel Signed-off-by: Tony Nguyen --- In the v3: addressed Paul's comment about commit header, - Change CH_MTP_I219_LM18 to PC

Re: [Intel-wired-lan] [PATCH v2] e1000e: move force SMBUS near the end of enable_ulp function

2024-05-17 Thread Hui Wang
On 5/17/24 18:07, Paul Menzel wrote: Dear Hui, Thank you for your response. Am 17.05.24 um 11:45 schrieb Hui Wang: On 5/17/24 13:45, Paul Menzel wrote: Am 08.05.24 um 14:06 schrieb Hui Wang: The commit 861e8086029e ("e1000e: move force SMBUS from enable ulp function to avoid PHY

Re: [Intel-wired-lan] [PATCH v2] e1000e: move force SMBUS near the end of enable_ulp function

2024-05-17 Thread Hui Wang
On 5/17/24 13:45, Paul Menzel wrote: Dear Hui, Thank you for your patch. Am 08.05.24 um 14:06 schrieb Hui Wang: The commit 861e8086029e ("e1000e: move force SMBUS from enable ulp function to avoid PHY loss issue") introduces a regression on CH_MTP_I219_LM18 (PCIID: 0x8086550A

[Intel-wired-lan] [PATCH v2] e1000e: move force SMBUS near the end of enable_ulp function

2024-05-08 Thread Hui Wang
mmediate ahead of hw->phy.ops.release(hw), this could allow the longest settling time as possible for interface in this function and doesn't change the original code logic. Fixes: 861e8086029e ("e1000e: move force SMBUS from enable ulp function to avoid PHY loss issue") Signed-off-by: Hui W

[Intel-wired-lan] [PATCH] e1000e: move force SMBUS near the end of enable_ulp function

2024-04-13 Thread Hui Wang
d of hw->phy.ops.release(hw), this could allow the longest settling time as possible for interface in this function and doesn't change the original code logic. Fixes: 861e8086029e ("e1000e: move force SMBUS from enable ulp function to avoid PHY loss issue") Signed-off-by: Hui Wang