crypto/cnxk: add PCI ID for cn9k

Message ID 1625812123-802-1-git-send-email-anoobj@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series crypto/cnxk: add PCI ID for cn9k |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/github-robot success github build: passed
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-abi-testing warning Testing issues
ci/iol-testing success Testing PASS

Commit Message

Anoob Joseph July 9, 2021, 6:28 a.m. UTC
  Add PCI ID for crypo_cn9k PMD.

To avoid conflicting PCI ID in crypto_octeontx2 and
crypto_cn9k PMDs, disable crypto_cn9k PMD when built with
octeontx2 config.

The lack of PCI ID is causing debug build to fail for
crypto_cn9k PMD.

Reported-by: Ali Alnubani <alialnu@mellanox.com>
Suggested-by: David Marchand <david.marchand@redhat.com>
Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
 config/arm/arm64_octeontx2_linux_gcc | 1 +
 drivers/crypto/cnxk/cn9k_cryptodev.c | 2 ++
 2 files changed, 3 insertions(+)
  

Comments

Akhil Goyal July 9, 2021, 6:30 a.m. UTC | #1
> Add PCI ID for crypo_cn9k PMD.
> 
> To avoid conflicting PCI ID in crypto_octeontx2 and
> crypto_cn9k PMDs, disable crypto_cn9k PMD when built with
> octeontx2 config.
> 
> The lack of PCI ID is causing debug build to fail for
> crypto_cn9k PMD.
> 
> Reported-by: Ali Alnubani <alialnu@mellanox.com>
> Suggested-by: David Marchand <david.marchand@redhat.com>
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> ---
Thomas,

Can you pick this directly on main?

Regards,
Akhil
  
David Marchand July 9, 2021, 12:09 p.m. UTC | #2
On Fri, Jul 9, 2021 at 8:29 AM Anoob Joseph <anoobj@marvell.com> wrote:
>
> Add PCI ID for crypo_cn9k PMD.
>
> To avoid conflicting PCI ID in crypto_octeontx2 and
> crypto_cn9k PMDs, disable crypto_cn9k PMD when built with
> octeontx2 config.
>
> The lack of PCI ID is causing debug build to fail for
> crypto_cn9k PMD.
>
> Reported-by: Ali Alnubani <alialnu@mellanox.com>
> Suggested-by: David Marchand <david.marchand@redhat.com>
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>

Applied, thanks.
  

Patch

diff --git a/config/arm/arm64_octeontx2_linux_gcc b/config/arm/arm64_octeontx2_linux_gcc
index 9156ee5..8fbdd38 100644
--- a/config/arm/arm64_octeontx2_linux_gcc
+++ b/config/arm/arm64_octeontx2_linux_gcc
@@ -14,3 +14,4 @@  endian = 'little'
 
 [properties]
 platform = 'octeontx2'
+disable_drivers = 'common/cnxk'
diff --git a/drivers/crypto/cnxk/cn9k_cryptodev.c b/drivers/crypto/cnxk/cn9k_cryptodev.c
index e74e739..71d144f 100644
--- a/drivers/crypto/cnxk/cn9k_cryptodev.c
+++ b/drivers/crypto/cnxk/cn9k_cryptodev.c
@@ -21,6 +21,8 @@  uint8_t cn9k_cryptodev_driver_id;
 
 static struct rte_pci_id pci_id_cpt_table[] = {
 	{
+		RTE_PCI_DEVICE(PCI_VENDOR_ID_CAVIUM,
+			       PCI_DEVID_CN9K_RVU_CPT_VF)
 	},
 	/* sentinel */
 	{