DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] net/igc: disable EEE by default on I225/226
@ 2023-06-20 14:56 Matthew Smith
  2023-06-21  0:52 ` Zhang, Qi Z
  0 siblings, 1 reply; 2+ messages in thread
From: Matthew Smith @ 2023-06-20 14:56 UTC (permalink / raw)
  To: junfeng.guo, simei.su; +Cc: dev, Matthew Smith

I226-V devices can hang when EEE is enabled. Explicitly disable
EEE during initialization of the device, as the Linux and FreeBSD
kernel drivers do.

Signed-off-by: Matthew Smith <mgsmith@netgate.com>
---
 drivers/net/igc/base/igc_i225.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/net/igc/base/igc_i225.c b/drivers/net/igc/base/igc_i225.c
index 180d3cf687..17a1573064 100644
--- a/drivers/net/igc/base/igc_i225.c
+++ b/drivers/net/igc/base/igc_i225.c
@@ -122,6 +122,9 @@ static s32 igc_init_mac_params_i225(struct igc_hw *hw)
 
 	mac->ops.write_vfta = igc_write_vfta_generic;
 
+	/* Disable EEE by default */
+	dev_spec->eee_disable = true;
+
 	return IGC_SUCCESS;
 }
 
@@ -1243,6 +1246,7 @@ s32 igc_init_hw_i225(struct igc_hw *hw)
 
 	hw->phy.ops.get_cfg_done = igc_get_cfg_done_i225;
 	ret_val = igc_init_hw_base(hw);
+	igc_set_eee_i225(hw, false, false, false);
 	return ret_val;
 }
 
-- 
2.30.1 (Apple Git-130)


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [PATCH] net/igc: disable EEE by default on I225/226
  2023-06-20 14:56 [PATCH] net/igc: disable EEE by default on I225/226 Matthew Smith
@ 2023-06-21  0:52 ` Zhang, Qi Z
  0 siblings, 0 replies; 2+ messages in thread
From: Zhang, Qi Z @ 2023-06-21  0:52 UTC (permalink / raw)
  To: Smith, Matthew, Guo, Junfeng, Su, Simei; +Cc: dev, Smith, Matthew



> -----Original Message-----
> From: Matthew Smith <mgsmith@netgate.com>
> Sent: Tuesday, June 20, 2023 10:56 PM
> To: Guo, Junfeng <junfeng.guo@intel.com>; Su, Simei <simei.su@intel.com>
> Cc: dev@dpdk.org; Smith, Matthew <mgsmith@netgate.com>
> Subject: [PATCH] net/igc: disable EEE by default on I225/226
> 
> I226-V devices can hang when EEE is enabled. Explicitly disable EEE during
> initialization of the device, as the Linux and FreeBSD kernel drivers do.
> 
> Signed-off-by: Matthew Smith <mgsmith@netgate.com>

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-06-21  0:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-20 14:56 [PATCH] net/igc: disable EEE by default on I225/226 Matthew Smith
2023-06-21  0:52 ` Zhang, Qi Z

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).