* [PATCH] net/ice: fix DCF init for E830 devices
@ 2024-07-19 13:44 Ian Stokes
2024-07-22 11:41 ` Bruce Richardson
0 siblings, 1 reply; 2+ messages in thread
From: Ian Stokes @ 2024-07-19 13:44 UTC (permalink / raw)
To: dev; +Cc: bruce.richardson, Ian Stokes
From: Bruce Richardson <bruce.richardson@intel.com>
E830 introduces a new version of Get Link Status Data
which increases the size of struct ice_aqc_get_link_status_data
from 32 bytes to 56. When initializing DCF, attempt to get
link status data using both formats of the command, by overriding
the mac type to E830 on failure with the default setting.
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Signed-off-by: Ian Stokes <ian.stokes@intel.com>
---
drivers/net/ice/ice_dcf_parent.c | 15 +++++++++++++--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ice/ice_dcf_parent.c b/drivers/net/ice/ice_dcf_parent.c
index 4e4a63fdd0..f92bd5e726 100644
--- a/drivers/net/ice/ice_dcf_parent.c
+++ b/drivers/net/ice/ice_dcf_parent.c
@@ -346,8 +346,19 @@ ice_dcf_init_parent_hw(struct ice_hw *hw)
/* Initialize port_info struct with link information */
status = ice_aq_get_link_info(hw->port_info, true, NULL, NULL);
- if (status)
- goto err_unroll_alloc;
+ if (status) {
+ enum ice_mac_type type = hw->mac_type;
+
+ /* DCF uses ICE_MAC_GENERIC which can be talking to either
+ * E810 or E830. Retry with E830 mac type to ensure correct
+ * data length is used for IAVF communication with PF.
+ */
+ hw->mac_type = ICE_MAC_E830;
+ status = ice_aq_get_link_info(hw->port_info, true, NULL, NULL);
+ hw->mac_type = type;
+ if (status)
+ goto err_unroll_alloc;
+ }
status = ice_init_fltr_mgmt_struct(hw);
if (status)
--
2.34.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] net/ice: fix DCF init for E830 devices
2024-07-19 13:44 [PATCH] net/ice: fix DCF init for E830 devices Ian Stokes
@ 2024-07-22 11:41 ` Bruce Richardson
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Richardson @ 2024-07-22 11:41 UTC (permalink / raw)
To: Ian Stokes; +Cc: dev
On Fri, Jul 19, 2024 at 02:44:27PM +0100, Ian Stokes wrote:
> From: Bruce Richardson <bruce.richardson@intel.com>
>
> E830 introduces a new version of Get Link Status Data
> which increases the size of struct ice_aqc_get_link_status_data
> from 32 bytes to 56. When initializing DCF, attempt to get
> link status data using both formats of the command, by overriding
> the mac type to E830 on failure with the default setting.
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> Signed-off-by: Ian Stokes <ian.stokes@intel.com>
> ---
> drivers/net/ice/ice_dcf_parent.c | 15 +++++++++++++--
> 1 file changed, 13 insertions(+), 2 deletions(-)
>
Patch applied to dpdk-next-net-intel
/Bruce
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-07-22 11:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-19 13:44 [PATCH] net/ice: fix DCF init for E830 devices Ian Stokes
2024-07-22 11:41 ` Bruce Richardson
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).