diff --git a/drivers/crypto/octeontx/otx_cryptodev_ops.c 
b/drivers/crypto/octeontx/otx_cryptodev_ops.c
index d25f9c1..cc0030e 100644
--- a/drivers/crypto/octeontx/otx_cryptodev_ops.c
+++ b/drivers/crypto/octeontx/otx_cryptodev_ops.c
@@ -10,9 +10,15 @@
  #include "cpt_pmd_logs.h"
#include "otx_cryptodev.h"
+#include "otx_cryptodev_capabilities.h"
  #include "otx_cryptodev_hw_access.h"
  #include "otx_cryptodev_ops.h"
+static const struct rte_cryptodev_capabilities otx_capabilities[] = {
+       OTX_SYM_CAPABILITIES,
+       RTE_CRYPTODEV_END_OF_CAPABILITIES_LIST()
+};
+

better to have otx_capabilities structure defined in the 
otx_cryptodev_capabilities.h

I don't see any value addition of creating a macro in one file using in a 
separate structure in another file

which doesn't have anything new in that structure. It would also give 
checkpatch error.

You can directly have a capability structure without the #define.

  /* Alarm routines */
static void

Reply via email to