* |WARNING| pw152576 [PATCH v2 3/8] net/ixgbe/base: correct definition of macro
[not found] <20250326160539.1316499-4-bruce.richardson@intel.com>
@ 2025-03-26 16:06 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2025-03-26 16:06 UTC (permalink / raw)
To: test-report; +Cc: Bruce Richardson
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/152576
_coding style issues_
ERROR:SPACING: space required before the open parenthesis '('
#116: FILE: drivers/net/intel/ixgbe/base/ixgbe_osdep.h:86:
+#define IXGBE_LE32_TO_CPUS(_i) do { *_i = rte_le_to_cpu_32(*_i); } while(0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_i' - possible side-effects?
#116: FILE: drivers/net/intel/ixgbe/base/ixgbe_osdep.h:86:
+#define IXGBE_LE32_TO_CPUS(_i) do { *_i = rte_le_to_cpu_32(*_i); } while(0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_i' may be better as '(_i)' to avoid precedence issues
#116: FILE: drivers/net/intel/ixgbe/base/ixgbe_osdep.h:86:
+#define IXGBE_LE32_TO_CPUS(_i) do { *_i = rte_le_to_cpu_32(*_i); } while(0)
WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#116: FILE: drivers/net/intel/ixgbe/base/ixgbe_osdep.h:86:
+#define IXGBE_LE32_TO_CPUS(_i) do { *_i = rte_le_to_cpu_32(*_i); } while(0)
total: 1 errors, 1 warnings, 2 checks, 32 lines checked
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2025-03-26 16:06 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20250326160539.1316499-4-bruce.richardson@intel.com>
2025-03-26 16:06 ` |WARNING| pw152576 [PATCH v2 3/8] net/ixgbe/base: correct definition of macro 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).