Hi Thomas Monjalon,

Thank you for your review and for guiding me on the mailing list workflow.

> Please could you send your new series as a reply of the v1 cover letter?
> It would avoid having a large indent in mail clients.

Got it. I will configure git send-email with "--in-reply-to" bound to the 
v1 cover letter Message-ID for the next version of this patch set. 
Apologies for the inconvenience caused by the deep threading.

> Also I'm still waiting for a reply to my question about the need
> for drivers/common/sxe2/.
> I'm not going to pull any sxe2 patch into main until you explain.

Regarding the necessity of the "drivers/common/sxe2/" directory, the sxe2 
hardware architecture is designed to support both standard NIC and vDPA 
(vHost Data Path Acceleration) functionalities. 

The code currently placed in "drivers/common/sxe2/" contains the foundational 
Hardware Abstraction Layer (HAL), firmware interfaces (AdminQ), and common 
register definitions. 

Apart from the "net/sxe2" PMD in this series, we are actively implementing the 
"vdpa/sxe2" driver, which will be submitted in a subsequent patch set. Both the 
"net/sxe2" and "vdpa/sxe2" drivers strictly depend on this shared common HAL 
layer to manage the same hardware core. Separating this code into 
"drivers/common/" 
is essential to prevent massive code duplication between the net and vdpa 
subsystems in the near future.

Please let me know if this clarifies your concern. Once confirmed, I will 
send out the updated v15 series as a reply to the v1 cover letter immediately.

Best regards,
Jie Liu ([email protected])
 


[email protected]
 
From: Thomas Monjalon
Date: 2026-06-09 16:42
To: Jie Liu
CC: stephen; dev
Subject: Re: [PATCH v14 00/20] net/sxe2: added Linkdata sxe2 ethernet driver
09/06/2026 03:39, [email protected]: 
> From: Jie Liu <[email protected]> 
> 
> This patch set implements core functionality for the SXE2 PMD, 
> including basic driver framework, data path setup, and advanced 
> offload features (VLAN, RSS,TM, PTP etc.). 
> 
> V14: 
>  - Addressed AI comments 

Please could you send your new series as a reply of the v1 cover letter? 
It would avoid having a large indent in mail clients. 

Also I'm still waiting for a reply to my question about the need 
for drivers/common/sxe2/. 
I'm not going to pull any sxe2 patch into main until you explain. 

Reply via email to