* [dpdk-test-report] |WARNING| pw28867 [PATCH v2 14/19] net/bnxt: add support for rx_queue_count
[not found] <20170918151755.86605-15-ajit.khaparde@broadcom.com>
@ 2017-09-18 15:20 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-09-18 15:20 UTC (permalink / raw)
To: test-report; +Cc: Ajit Khaparde
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/28867
_coding style issues_
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'cons' may be better as '(cons)' to avoid precedence issues
#43: FILE: drivers/net/bnxt/bnxt_cpr.h:54:
+#define FLIP_VALID(cons, mask, val) (cons >= mask ? !val : val)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'mask' may be better as '(mask)' to avoid precedence issues
#43: FILE: drivers/net/bnxt/bnxt_cpr.h:54:
+#define FLIP_VALID(cons, mask, val) (cons >= mask ? !val : val)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'val' - possible side-effects?
#43: FILE: drivers/net/bnxt/bnxt_cpr.h:54:
+#define FLIP_VALID(cons, mask, val) (cons >= mask ? !val : val)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'val' may be better as '(val)' to avoid precedence issues
#43: FILE: drivers/net/bnxt/bnxt_cpr.h:54:
+#define FLIP_VALID(cons, mask, val) (cons >= mask ? !val : val)
total: 0 errors, 0 warnings, 4 checks, 118 lines checked
^ permalink raw reply [flat|nested] only message in thread