[dpdk-test-report] |WARNING| [PATCH 29/32] net/dpaa2: rx packet parsing and packet type support

checkpatch at dpdk.org checkpatch at dpdk.org
Sun Dec 4 13:51:28 CET 2016


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

_coding style issues_


WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#239: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:85:
+	/**<	word5:
+		ShimOffset_1 + ShimOffset_2 + IPPIDOffset + EthOffset +

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#246: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:92:
+	/**<	word6:
+		PPPoEOffset + MPLSOffset_1 + MPLSOffset_n + ARPorIPOffset_1

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#253: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:99:
+	/**<	word7:
+		RoutingHdrOfset1 + RoutingHdrOfset2 + NxtHdrOffset + IPv6FragOffset +

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#259: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:105:
+	/**<	word8:
+		ParseErrorcode + Soft Parsing Context (1 + 7 bytes)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#271: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:117:
+#define BIT_SET_AT_POS(var, pos)	(var |= (pos))

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#273: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:119:
+#define BIT_RESET_AT_POS(var, pos)	(var &= ~(pos))

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#275: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:121:
+#define BIT_ISSET_AT_POS(var, pos)	((var & (pos)) ? 1 : 0)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#279: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:125:
+#define NEXT_HDR(var)			((uint64_t)var & 0xFFFF000000000000)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#281: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:127:
+#define FAF_EXTN_RESERVED(var)		((uint64_t)var & 0x00007FFF00000000)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'var' may be better as '(var)' to avoid precedence issues
#282: FILE: drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h:128:
+#define FAF_USER_DEFINED_RESERVED(var)	((uint64_t)var & 0x00000000FF000000)

total: 0 errors, 4 warnings, 6 checks, 404 lines checked


More information about the test-report mailing list