<div dir="ltr">Thank you Ferruh,<div>I understand now.</div><div>We will abandon this patch and do exactly as you suggest with two separate patches.</div><div>Thanks for your guidance. Look for the two patches ETA tomorrow.</div><div>-Shep</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 10, 2023 at 5:56 PM Ferruh Yigit <<a href="mailto:ferruh.yigit@amd.com">ferruh.yigit@amd.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2/10/2023 10:35 PM, Shepard Siegel wrote:<br>
> Add ark PCIe device 1d6c:1022 FX2 to pci_id_ark_map.<br>
> Include introduced FX2 PCIe ID and description.<br>
> <br>
<br>
Thanks for v2, but this is no more just a documentation patch,<br>
can you please split the patch into two,<br>
first one adds new device support and update documentation related to<br>
new device, with a 'net/ark:' title,<br>
second one updates the document for extended information which is most<br>
of this patch<br>
<br>
<br>
> Signed-off-by: Shepard Siegel <<a href="mailto:shepard.siegel@atomicrules.com" target="_blank">shepard.siegel@atomicrules.com</a>><br>
> ---<br>
> doc/guides/nics/ark.rst | 20 ++++++++++++++++++++<br>
> drivers/net/ark/ark_ethdev.c | 1 +<br>
> 2 files changed, 21 insertions(+)<br>
> <br>
> diff --git a/doc/guides/nics/ark.rst b/doc/guides/nics/ark.rst<br>
> index ba00f14e80..39cd75064d 100644<br>
> --- a/doc/guides/nics/ark.rst<br>
> +++ b/doc/guides/nics/ark.rst<br>
> @@ -52,6 +52,10 @@ board. While specific capabilities such as number of physical<br>
> hardware queue-pairs are negotiated; the driver is designed to<br>
> remain constant over a broad and extendable feature set.<br>
> <br>
> +* FPGA Vendors Supported: AMD/Xilinx and Intel<br>
> +* Number of RX/TX Queue-Pairs: up to 128<br>
> +* PCIe Endpoint Technology: Gen3, Gen4, Gen5<br>
> +<br>
> Intentionally, Arkville by itself DOES NOT provide common NIC<br>
> capabilities such as offload or receive-side scaling (RSS).<br>
> These capabilities would be viewed as a gate-level "tax" on<br>
> @@ -302,6 +306,20 @@ ARK PMD supports the following Arkville RTL PCIe instances including:<br>
> * ``1d6c:101c`` - AR-ARK-SRIOV-VF [Arkville Virtual Function]<br>
> * ``1d6c:101e`` - AR-ARKA-FX1 [Arkville 64B DPDK Data Mover for Agilex R-Tile]<br>
> * ``1d6c:101f`` - AR-TK242 [2x100GbE Packet Capture Device]<br>
> +* ``1d6c:1022`` - AR-ARKA-FX2 [Arkville 128B DPDK Data Mover for Agilex]<br>
> +<br>
> +Arkville RTL Core Configurations<br>
> +-------------------------------------<br>
> +<br>
> +Arkville's RTL core may be configured by the user for three different<br>
> +datapath widths to balance throughput against FPGA logic area. The ARK PMD<br>
> +has introspection on the RTL core configuration and acts accordingly.<br>
> +All three configurations present identical RTL user-facing AXI stream<br>
> +interfaces for both AMD/Xilinx and Intel FPGAs.<br>
> +<br>
> +* ARK-FX0 - 256-bit 32B datapath (PCIe Gen3, Gen4)<br>
> +* ARK-FX1 - 512-bit 64B datapath (PCIe Gen3, Gen4, Gen5)<br>
> +* ARK-FX2 - 1024-bit 128B datapath (PCIe Gen5x16 Only)<br>
> <br>
> DPDK and Arkville Firmware Versioning<br>
> -------------------------------------<br>
> @@ -334,6 +352,8 @@ Supported Features<br>
> ------------------<br>
> <br>
> * Dynamic ARK PMD extensions<br>
> +* Dynamic per-queue MBUF (re)sizing up to 32KB<br>
> +* SR-IOV, VF-based queue-segregation<br>
> * Multiple receive and transmit queues<br>
> * Jumbo frames up to 9K<br>
> * Hardware Statistics<br>
> diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c<br>
> index c654a229f7..b2995427c8 100644<br>
> --- a/drivers/net/ark/ark_ethdev.c<br>
> +++ b/drivers/net/ark/ark_ethdev.c<br>
> @@ -99,6 +99,7 @@ static const struct rte_pci_id pci_id_ark_map[] = {<br>
> {RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101c)},<br>
> {RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101e)},<br>
> {RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101f)},<br>
> + {RTE_PCI_DEVICE(AR_VENDOR_ID, 0x1022)},<br>
> {.vendor_id = 0, /* sentinel */ },<br>
> };<br>
> <br>
<br>
</blockquote></div>