[PATCH] app/testpmd: fix compilation with I40E disabled

[PATCH] app/testpmd: fix compilation with I40E disabled

[Kirill Rybalchenko]

Precompiler instructions #ifdef RTE_LIBRTE_I40E_PMD ... #endif
were not placed correctly, which caused number of
compilation errors if I40E PMD is disabled.

Fixes: 5a4806d304e0 ("app/testpmd: support updating pctype mapping")

Signed-off-by: Kirill Rybalchenko <kirill.rybalchenko@intel.com>
---
 app/test-pmd/cmdline.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index 20e04f7..8dc5c85 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -13843,8 +13843,8 @@ cmd_ddp_info_parsed(
 
 	free(proto);
 	ret = 0;
-#endif
 no_print_return:
+#endif
 	if (ret == -ENOTSUP)
 		printf("Function not supported in PMD driver\n");
 	close_ddp_package_file(pkg);
@@ -14390,17 +14390,16 @@ cmd_pctype_mapping_update_parsed(
 #ifdef RTE_LIBRTE_I40E_PMD
 	struct rte_pmd_i40e_flow_type_mapping mapping;
 	unsigned int i;
-#endif
 	unsigned int nb_item;
 	unsigned int pctype_list[RTE_PMD_I40E_PCTYPE_MAX];
+#endif
 
 	if (port_id_is_invalid(res->port_id, ENABLED_WARN))
 		return;
 
+#ifdef RTE_LIBRTE_I40E_PMD
 	nb_item = parse_item_list(res->pctype_list, "pctypes",
 				  RTE_PMD_I40E_PCTYPE_MAX, pctype_list, 1);
-
-#ifdef RTE_LIBRTE_I40E_PMD
 	mapping.flow_type = res->flow_type;
 	for (i = 0, mapping.pctype = 0ULL; i < nb_item; i++)
 		mapping.pctype |= (1ULL << pctype_list[i]);
-- 
2.5.5

Re: [PATCH] app/testpmd: fix compilation with I40E disabled

[Ferruh Yigit]

On 10/9/2017 3:38 PM, Kirill Rybalchenko wrote:
> Precompiler instructions #ifdef RTE_LIBRTE_I40E_PMD ... #endif
> were not placed correctly, which caused number of
> compilation errors if I40E PMD is disabled.
> 
> Fixes: 5a4806d304e0 ("app/testpmd: support updating pctype mapping")
> 
> Signed-off-by: Kirill Rybalchenko <kirill.rybalchenko@intel.com>

Applied to dpdk-next-net/master, thanks.