> -----Original Message----- > From: zbigniew.bo...@caviumnetworks.com > [mailto:zbigniew.bo...@caviumnetworks.com] > Sent: Wednesday, December 07, 2016 2:33 AM > To: De Lara Guarch, Pablo; jerin.ja...@caviumnetworks.com > Cc: dev@dpdk.org; Zbigniew Bodek > Subject: [PATCH v2 08/12] mk/crypto/armv8: add PMD to the build system > > From: Zbigniew Bodek <zbigniew.bo...@caviumnetworks.com> > > Build ARMv8 crypto PMD if compiling for ARM64 > and CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO option > is enable in the configuration file. > > Signed-off-by: Zbigniew Bodek <zbigniew.bo...@caviumnetworks.com> > --- > drivers/crypto/Makefile | 3 +++ > mk/rte.app.mk | 3 +++ > 2 files changed, 6 insertions(+) > > diff --git a/drivers/crypto/Makefile b/drivers/crypto/Makefile > index 745c614..a5de944 100644 > --- a/drivers/crypto/Makefile > +++ b/drivers/crypto/Makefile > @@ -33,6 +33,9 @@ include $(RTE_SDK)/mk/rte.vars.mk > > DIRS-$(CONFIG_RTE_LIBRTE_PMD_AESNI_GCM) += aesni_gcm > DIRS-$(CONFIG_RTE_LIBRTE_PMD_AESNI_MB) += aesni_mb > +ifeq ($(CONFIG_RTE_ARCH_ARM64),y) > +DIRS-$(CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO) += armv8 > +endif
Is this extra conditional necessary (ARM64)? I would think enabling ARMV8_CRYPTO would be sufficient.