[DPDK] app/testpmd: fix unused variable compile error

Message ID 20190925175400.62253-1-taox.zhu@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers
Series [DPDK] app/testpmd: fix unused variable compile error |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail apply issues

Commit Message

Zhu, TaoX Sept. 25, 2019, 5:54 p.m. UTC
  From: Zhu Tao <taox.zhu@intel.com>

This minor patch fixes unused variable ‘ret’ compile error When
CONFIG_RTE_LIBRTE_I40E_16BYTE_RX_DESC was selected.

Fixes: f5267e485a2a ("app/testpmd: check status of getting ethdev info")

Signed-off-by: Zhu Tao <taox.zhu@intel.com>
---
 app/test-pmd/config.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Andrew Rybchenko Sept. 25, 2019, 10:34 a.m. UTC | #1
On 9/25/19 8:54 PM, taox.zhu@intel.com wrote:
> From: Zhu Tao <taox.zhu@intel.com>
>
> This minor patch fixes unused variable ‘ret’ compile error When
> CONFIG_RTE_LIBRTE_I40E_16BYTE_RX_DESC was selected.
>
> Fixes: f5267e485a2a ("app/testpmd: check status of getting ethdev info")
>
> Signed-off-by: Zhu Tao <taox.zhu@intel.com>

Reviewed-by: Andrew Rybchenko <arybchenko@solarflare.com>

Thanks.
  

Patch

diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
index 957c61fbe..a3b6cbd08 100644
--- a/app/test-pmd/config.c
+++ b/app/test-pmd/config.c
@@ -1649,10 +1649,10 @@  ring_rx_descriptor_display(const struct rte_memzone *ring_mz,
 #endif
 			   uint16_t desc_id)
 {
-	int ret;
 	struct igb_ring_desc_16_bytes *ring =
 		(struct igb_ring_desc_16_bytes *)ring_mz->addr;
 #ifndef RTE_LIBRTE_I40E_16BYTE_RX_DESC
+	int ret;
 	struct rte_eth_dev_info dev_info;
 
 	ret = eth_dev_info_get_print_err(port_id, &dev_info);