DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] armv7a: disable qede pmd
@ 2016-05-09 21:34 Jan Viktorin
  2016-05-13 15:01 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Viktorin @ 2016-05-09 21:34 UTC (permalink / raw)
  To: dev; +Cc: Rasesh Mody, Thomas Monjalon, Jan Viktorin

The qede PMD driver is failing when building for ARMv7:

drivers/net/qede/base/ecore_dev.c: In function ‘ecore_hw_init_pf’:
drivers/net/qede/base/ecore_dev.c:1150:6:
	error: variable ‘prs_reg’ set but not used [-Werror=unused-but-set-variable]
  u32 prs_reg;
      ^
[...]

drivers/net/qede/base/ecore_dev.c:2492:13:
	error: variable ‘p_phys’ set but not used [-Werror=unused-but-set-variable]
  dma_addr_t p_phys = p_chain->p_phys_addr, p_phys_next = 0;
             ^
drivers/net/qede/base/ecore_dev.c: In function ‘ecore_chain_free_pbl’:
drivers/net/qede/base/ecore_dev.c:2532:39:
	error: variable ‘pbl_size’ set but not used [-Werror=unused-but-set-variable]
  u32 page_cnt = p_chain->page_cnt, i, pbl_size;

Fixes: 3eae93a9bfd5 ("qede: enable PMD build")

Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
---
 config/defconfig_arm-armv7a-linuxapp-gcc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/config/defconfig_arm-armv7a-linuxapp-gcc b/config/defconfig_arm-armv7a-linuxapp-gcc
index e626ec1..bde6acd 100644
--- a/config/defconfig_arm-armv7a-linuxapp-gcc
+++ b/config/defconfig_arm-armv7a-linuxapp-gcc
@@ -74,3 +74,4 @@ CONFIG_RTE_LIBRTE_MPIPE_PMD=n
 CONFIG_RTE_LIBRTE_VMXNET3_PMD=n
 CONFIG_RTE_LIBRTE_PMD_XENVIRT=n
 CONFIG_RTE_LIBRTE_PMD_BNX2X=n
+CONFIG_RTE_LIBRTE_QEDE_PMD=n
-- 
2.6.2

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] armv7a: disable qede pmd
  2016-05-09 21:34 [dpdk-dev] [PATCH] armv7a: disable qede pmd Jan Viktorin
@ 2016-05-13 15:01 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2016-05-13 15:01 UTC (permalink / raw)
  To: Jan Viktorin; +Cc: dev, Rasesh Mody

2016-05-09 23:34, Jan Viktorin:
> The qede PMD driver is failing when building for ARMv7:
> 
> drivers/net/qede/base/ecore_dev.c: In function ‘ecore_hw_init_pf’:
> drivers/net/qede/base/ecore_dev.c:1150:6:
> 	error: variable ‘prs_reg’ set but not used [-Werror=unused-but-set-variable]
>   u32 prs_reg;
>       ^
> [...]
> 
> drivers/net/qede/base/ecore_dev.c:2492:13:
> 	error: variable ‘p_phys’ set but not used [-Werror=unused-but-set-variable]
>   dma_addr_t p_phys = p_chain->p_phys_addr, p_phys_next = 0;
>              ^
> drivers/net/qede/base/ecore_dev.c: In function ‘ecore_chain_free_pbl’:
> drivers/net/qede/base/ecore_dev.c:2532:39:
> 	error: variable ‘pbl_size’ set but not used [-Werror=unused-but-set-variable]
>   u32 page_cnt = p_chain->page_cnt, i, pbl_size;
> 
> Fixes: 3eae93a9bfd5 ("qede: enable PMD build")
> 
> Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>

Applied, not because of the build issue, but because ARMv7 is
not really interested in PCI devices.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-05-13 15:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-09 21:34 [dpdk-dev] [PATCH] armv7a: disable qede pmd Jan Viktorin
2016-05-13 15:01 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).