automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw32399 [PATCH 10/11] qede: implement dynamic logging
       [not found] <20171219063840.18981-11-stephen@networkplumber.org>
@ 2017-12-19  6:40 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-12-19  6:40 UTC (permalink / raw)
  To: test-report; +Cc: Stephen Hemminger

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/32399

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p_dev' - possible side-effects?
#83: FILE: drivers/net/qede/qede_logs.h:14:
+#define DP_ERR(p_dev, fmt, ...)				\
+	rte_log(RTE_LOG_ERR, qede_logtype_driver,	\
+		"[%s:%d(%s)]" fmt,			\
+		__func__, __LINE__,			\
+		(p_dev)->name ? (p_dev)->name : "",	\
 		##__VA_ARGS__)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p_dev' - possible side-effects?
#131: FILE: drivers/net/qede/qede_logs.h:48:
+#define DP_VERBOSE(p_dev, module, fmt, ...)				\
+	do {								\
+		if ((p_dev)->dp_module & module)			\
+			rte_log(RTE_LOG_DEBUG, qede_logtype_driver,	\
+				"[%s:%d(%s)]" fmt,			\
+				__func__, __LINE__,			\
+				(p_dev)->name ? (p_dev)->name : "",	\
+				##__VA_ARGS__);				\
+	} while (0)

total: 0 errors, 0 warnings, 2 checks, 117 lines checked

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-12-19  6:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20171219063840.18981-11-stephen@networkplumber.org>
2017-12-19  6:40 ` [dpdk-test-report] |WARNING| pw32399 [PATCH 10/11] qede: implement dynamic logging checkpatch

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).