fm10k: fix build with unused debug function
authorJeff Shaw <jeffrey.b.shaw@intel.com>
Wed, 18 Feb 2015 18:07:40 +0000 (10:07 -0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 24 Feb 2015 02:09:57 +0000 (03:09 +0100)
This commit fixes the following error which was reported when
compiling with clang by moving the function inside an
RTE_LIBRTE_FM10K_DEBUG_RX ifdef block.

error: unused function 'dump_rxd'

Signed-off-by: Jeff Shaw <jeffrey.b.shaw@intel.com>
Acked-by: Keith Wiles <keith.wiles@intel.com>
lib/librte_pmd_fm10k/fm10k_rxtx.c

index f6768f0..83bddfc 100644 (file)
 #define rte_packet_prefetch(p)  do {} while (0)
 #endif
 
+#ifdef RTE_LIBRTE_FM10K_DEBUG_RX
 static inline void dump_rxd(union fm10k_rx_desc *rxd)
 {
-#ifndef RTE_LIBRTE_FM10K_DEBUG_RX
-       RTE_SET_USED(rxd);
-#endif
        PMD_RX_LOG(DEBUG, "+----------------|----------------+");
        PMD_RX_LOG(DEBUG, "|     GLORT      | PKT HDR & TYPE |");
        PMD_RX_LOG(DEBUG, "|   0x%08x   |   0x%08x   |", rxd->d.glort,
@@ -62,6 +60,7 @@ static inline void dump_rxd(union fm10k_rx_desc *rxd)
        PMD_RX_LOG(DEBUG, "|       0x%016lx        |", rxd->q.timestamp);
        PMD_RX_LOG(DEBUG, "+----------------|----------------+");
 }
+#endif
 
 static inline void
 rx_desc_to_ol_flags(struct rte_mbuf *m, const union fm10k_rx_desc *d)