Re: [PATCH v1] nics: add E2200 to the NIC guide

Bruce Richardson <[email protected]>
Newsgroups org.dpdk.dev
Message-ID <[email protected]>
On Wed, Aug 12, 2026 at 10:09:07AM +0800, Hailin Xu wrote:
> nics/cpfl: add E2200 to cpfi
> nics/idpf: add E2200 to idpf
> 

Needs a proper text description. The commit title probably should start
with "doc" or "doc/nics" rather than just "doc".

> Signed-off-by: Hailin Xu <[email protected]>
> ---
>  doc/guides/nics/cpfl.rst | 1 +
>  doc/guides/nics/idpf.rst | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/guides/nics/cpfl.rst b/doc/guides/nics/cpfl.rst
> index 3e2788ba03..5964a980ee 100644
> --- a/doc/guides/nics/cpfl.rst
> +++ b/doc/guides/nics/cpfl.rst
> @@ -8,6 +8,7 @@ CPFL Poll Mode Driver
>  
>  The cpfl PMD (**librte_net_cpfl**) provides poll mode driver support for
>  Intel\ |reg| Infrastructure Processing Unit (Intel\ |reg| IPU) E2100.
> +Intel\ |reg| Infrastructure Processing Unit (Intel\ |reg| IPU) E2200.

Since this is not a bulleted list or similar, best to add an "and" in
between the two NIC names.

>  Please refer to
>  https://www.intel.com/content/www/us/en/products/network-io/infrastructure-processing-units/asic/e2000-asic.html
>  for more information.
> diff --git a/doc/guides/nics/idpf.rst b/doc/guides/nics/idpf.rst
> index f4cd69ba7d..1f06eed645 100644
> --- a/doc/guides/nics/idpf.rst
> +++ b/doc/guides/nics/idpf.rst
> @@ -8,7 +8,7 @@ IDPF Poll Mode Driver
>  
>  The idpf PMD (**librte_net_idpf**) provides poll mode driver support for
>  Intel\ |reg| Infrastructure Processing Unit (Intel\ |reg| IPU) E2100.
> -
> +Intel\ |reg| Infrastructure Processing Unit (Intel\ |reg| IPU) E2200.
>  

Similarly here. However, I think in this case you probably want to keep the
two blank lines at end of the section i.e. just adding a new line rather
than replacing a blank line with the new text.

>  Linux Prerequisites
>  -------------------
> -- 
> 2.34.1
> 
Regards,
/Bruce
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.