On 2/18/15, 12:07 PM, "Shaw, Jeffrey B" <jeffrey.b.shaw at intel.com> wrote:
>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 at intel.com> Acked-by: keith.wiles at intel.com >--- > lib/librte_pmd_fm10k/fm10k_rxtx.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > >diff --git a/lib/librte_pmd_fm10k/fm10k_rxtx.c >b/lib/librte_pmd_fm10k/fm10k_rxtx.c >index f6768f0..83bddfc 100644 >--- a/lib/librte_pmd_fm10k/fm10k_rxtx.c >+++ b/lib/librte_pmd_fm10k/fm10k_rxtx.c >@@ -41,11 +41,9 @@ > #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) >-- >2.1.0 >