[dpdk-dev] [PATCH] crypto/mrvl: add missing library dependencies

[dpdk-dev] [PATCH] crypto/mrvl: add missing library dependencies

[Tomasz Duszynski]

While trying to do a shared build one will get linkage error
since a couple of library dependencies are missing from a makefile.

At some point there was a batch update of all PMDs but mrvl crypto was
missed back then.

Necessary makefile changes were introduced in
commit cbc12b0a96f5 ("mk: do not generate LDLIBS from directory dependencies")

Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
---
 drivers/crypto/mrvl/Makefile | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/crypto/mrvl/Makefile b/drivers/crypto/mrvl/Makefile
index bc5c227..9870f02 100644
--- a/drivers/crypto/mrvl/Makefile
+++ b/drivers/crypto/mrvl/Makefile
@@ -58,6 +58,8 @@ EXPORT_MAP := rte_pmd_mrvl_version.map
 
 # external library dependencies
 LDLIBS += -L$(LIBMUSDK_PATH)/lib -lmusdk
+LDLIBS += -lrte_eal -lrte_mbuf -lrte_mempool
+LDLIBS += -lrte_cryptodev
 LDLIBS += -lrte_bus_vdev
 
 # library source files
-- 
2.7.4

Re: [dpdk-dev] [PATCH] crypto/mrvl: add missing library dependencies

[De Lara Guarch, Pablo]

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Tomasz Duszynski
> Sent: Wednesday, March 21, 2018 9:45 AM
> To: dev@dpdk.org
> Cc: dima@marvell.com; nsamsono@marvell.com; jck@semihalf.com;
> jianbo.liu@arm.com; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>; stable@dpdk.org; Tomasz Duszynski
> <tdu@semihalf.com>
> Subject: [dpdk-dev] [PATCH] crypto/mrvl: add missing library dependencies
> 
> While trying to do a shared build one will get linkage error since a couple of
> library dependencies are missing from a makefile.
> 
> At some point there was a batch update of all PMDs but mrvl crypto was
> missed back then.
> 
> Necessary makefile changes were introduced in commit cbc12b0a96f5
> ("mk: do not generate LDLIBS from directory dependencies")
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>

Applied to dpdk-next-crypto.
Thanks,

Pablo