[dpdk-test-report] |WARNING| pw28867 [PATCH v2 14/19] net/bnxt: add support for rx_queue_count

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Sep 18 17:20:11 CEST 2017


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


More information about the test-report mailing list