<div dir="ltr">Hi Ferruh,<div><br></div><div>If you dont mind squashing 2/4 3/4 4/4 together, that would be great.</div><div><br></div><div>Thank you for correcting on the allowlist, s..b "supported PCIe ids list" instead.</div><div><br></div><div>best, Shep</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 13, 2023 at 10:51 AM 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/13/2023 2:58 PM, Shepard Siegel wrote:<br>
> This patch adds the Arkville FX2 device to the PCIe allowlist.<br>
> <br>
> Signed-off-by: Shepard Siegel <<a href="mailto:shepard.siegel@atomicrules.com" target="_blank">shepard.siegel@atomicrules.com</a>><br>
> ---<br>
>  drivers/net/ark/ark_ethdev.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
> <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>
It is possible to squash 2/4, 3/4 & 4/4 into single patch, I can do it<br>
while merging.<br>
<br>
In the patch title it mentions 'PCIe allowlist', this is not a kind of<br>
allow list, this is list of PCI ids supported by driver, so I can update<br>
patch title as following while merging if there won't be a new version:<br>
"net/ark: support Arkville FX2 device"<br>
</blockquote></div>