diff mbox series

[1/2] net/ark: add new device to PCIe allowlist

Message ID 20230211141459.2857045-1-shepard.siegel@atomicrules.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers
Series [1/2] net/ark: add new device to PCIe allowlist |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation warning apply patch failure
ci/iol-testing warning apply patch failure

Commit Message

Shepard Siegel Feb. 11, 2023, 2:14 p.m. UTC 
  This patch adds the Arkville FX2 device to the PCIe allowlist.

Signed-off-by: Shepard Siegel <shepard.siegel@atomicrules.com>
---
 drivers/net/ark/ark_ethdev.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Ferruh Yigit Feb. 13, 2023, 1:46 p.m. UTC | #1 
On 2/11/2023 2:14 PM, Shepard Siegel wrote:
> This patch adds the Arkville FX2 device to the PCIe allowlist.
> 
> Signed-off-by: Shepard Siegel <shepard.siegel@atomicrules.com>

Hi Shepard, Ed,

The patchset looks good, I can squash them into single patch while merging.

I don't know how major this new device is, if it is an important one you
may want to introduce it in the release notes too, what do think?

And if you will send a new version, can you please put previous doc
patch [1] and this to the same patchset since this one depends previous
doc patch to apply cleanly.

Thanks.


[1]
doc: clarify the existing net/ark guide
Shepard Siegel Feb. 13, 2023, 3:02 p.m. UTC | #2 
Thank you Ferruh,

We have rolled all changes, including your suggestion on the Release Note,
into the four distinct patches in this patchset which collectively brings
the
ark PMD up to date for this new capability.

best, Shep


On Mon, Feb 13, 2023 at 8:46 AM Ferruh Yigit <ferruh.yigit@amd.com> wrote:

> On 2/11/2023 2:14 PM, Shepard Siegel wrote:
> > This patch adds the Arkville FX2 device to the PCIe allowlist.
> >
> > Signed-off-by: Shepard Siegel <shepard.siegel@atomicrules.com>
>
> Hi Shepard, Ed,
>
> The patchset looks good, I can squash them into single patch while merging.
>
> I don't know how major this new device is, if it is an important one you
> may want to introduce it in the release notes too, what do think?
>
> And if you will send a new version, can you please put previous doc
> patch [1] and this to the same patchset since this one depends previous
> doc patch to apply cleanly.
>
> Thanks.
>
>
> [1]
> doc: clarify the existing net/ark guide
>
diff mbox series

Patch

diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c
index c654a229f7..b2995427c8 100644
--- a/drivers/net/ark/ark_ethdev.c
+++ b/drivers/net/ark/ark_ethdev.c
@@ -99,6 +99,7 @@  static const struct rte_pci_id pci_id_ark_map[] = {
 	{RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101c)},
 	{RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101e)},
 	{RTE_PCI_DEVICE(AR_VENDOR_ID, 0x101f)},
+	{RTE_PCI_DEVICE(AR_VENDOR_ID, 0x1022)},
 	{.vendor_id = 0, /* sentinel */ },
 };