On 12/3/2016 2:43 AM, Harish Patil wrote: > From: Rasesh Mody <rasesh.m...@cavium.com> > > Replace CONFIG_RTE_LIBRTE_QEDE_DEBUG_DRIVER with > CONFIG_RTE_LIBRTE_QEDE_DEBUG_VAL which is a 32-bit bitmapped value > where each bit represent a particular submodule to debug. Also move > notice messages under CONFIG_RTE_LIBRTE_QEDE_DEBUG_INFO. > > Signed-off-by: Harish Patil <harish.pa...@qlogic.com> > Signed-off-by: Rasesh Mody <rasesh.m...@cavium.com> > ---
Is 32bit supported by driver? If so it is throwing a compilation error for it [1], if not can you please document it? [1] In file included from .../drivers/net/qede/base/ecore.h:35:0, from .../drivers/net/qede/qede_ethdev.h:22, from .../drivers/net/qede/qede_ethdev.c:9: .../drivers/net/qede/qede_ethdev.c: In function ‘qede_rss_hash_update’: .../drivers/net/qede/base/../qede_logs.h:33:3: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 7 has type ‘uint64_t {aka long long unsigned int}’ [-Werror=format=] "[%s:%d(%s)]" fmt, \ ^ .../drivers/net/qede/qede_ethdev.c:1472:2: note: in expansion of macro ‘DP_INFO’ DP_INFO(edev, "RSS hf = 0x%lx len = %u key = %p\n", hf, len, key); ^~~~~~~ cc1: all warnings being treated as errors <...>