[dpdk-test-report] |WARNING| pw72736 [PATCH v3 15/51] net/bnxt: add HCAPI interface support

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jul 2 06:15:10 CEST 2020


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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'mask' - possible side-effects?
#511: FILE: drivers/net/bnxt/hcapi/hcapi_cfa_defs.h:26:
+#define __CFA_ALIGN_MASK(x, mask) (((x) + (mask)) & ~(mask))

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#561: FILE: drivers/net/bnxt/hcapi/hcapi_cfa_defs.h:76:
+	HCAPI_CFA_HWOPS_ADD, /**< For operations which require more than simple
+			      * writes to HW, this operation is used. The

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#569: FILE: drivers/net/bnxt/hcapi/hcapi_cfa_defs.h:84:
+	HCAPI_CFA_HWOPS_DEL, /**< This issues operations to clear the hardware.
+			      * This operation is used in conjunction

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'crc' - possible side-effects?
#1215: FILE: drivers/net/bnxt/hcapi/hcapi_cfa_p4.c:52:
+#define ucrc32(ch, crc) (crc32tbl[((crc) ^ (ch)) & 0xff] ^ ((crc) >> 8))

WARNING:LONG_LINE: line over 90 characters
#2089: FILE: drivers/net/bnxt/tf_core/tf_em.c:387:
+	mask = tf_em_get_key_mask(tbl_scope_cb->em_ctx_info[parms->dir].em_tables[TF_KEY0_TABLE].num_entries);

total: 0 errors, 3 warnings, 2 checks, 2228 lines checked


More information about the test-report mailing list