qede: add missing external dependency and disable by default
authorPanu Matilainen <pmatilai@redhat.com>
Tue, 24 May 2016 10:28:31 +0000 (13:28 +0300)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 24 May 2016 14:17:02 +0000 (16:17 +0200)
The qede driver depends on libz but the LDLIBS entry in makefile
was missing. Also because of the external dependency, make it
disabled in default config as per common DPDK policy on external deps.

Fixes: ec94dbc57362 ("qede: add base driver")

Signed-off-by: Panu Matilainen <pmatilai@redhat.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
config/common_base
drivers/net/qede/Makefile

index 3535c6e..47c26f6 100644 (file)
@@ -299,7 +299,7 @@ CONFIG_RTE_LIBRTE_BOND_DEBUG_ALB_L1=n
 
 # QLogic 25G/40G PMD
 #
-CONFIG_RTE_LIBRTE_QEDE_PMD=y
+CONFIG_RTE_LIBRTE_QEDE_PMD=n
 CONFIG_RTE_LIBRTE_QEDE_DEBUG_INIT=n
 CONFIG_RTE_LIBRTE_QEDE_DEBUG_INFO=n
 CONFIG_RTE_LIBRTE_QEDE_DEBUG_DRV=n
index 4cc9ee8..c9b3b1c 100644 (file)
@@ -14,6 +14,8 @@ LIB = librte_pmd_qede.a
 CFLAGS += -O3
 CFLAGS += $(WERROR_FLAGS)
 
+LDLIBS += -lz
+
 EXPORT_MAP := rte_pmd_qede_version.map
 
 LIBABIVER := 1