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:
it
immediate 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
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). W
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
ce SMBUS from 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_
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,
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
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 w
.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/
Hi Vitaly
I tested the patch on a laptop with the ethernet card [8086:550A], the
system suspend and resume worked well with the cable plugged or unplugged.
But I still think that reverting 2 regression commits is a better solution.
Thanks.
And some comment below:
On 6/13/24 20:01, Vitaly Li
On 6/19/24 20:17, Lifshits, Vitaly wrote:
Hi Hui,
On 6/14/2024 4:53 AM, Hui Wang wrote:
Hi Vitaly
I tested the patch on a laptop with the ethernet card [8086:550A],
the system suspend and resume worked well with the cable plugged or
unplugged.
But I still think that reverting 2
11 matches
Mail list logo
