[dpdk-test-report] |WARNING| pw25804 [PATCH v4 01/17] net/i40e/base: use new virtchnl header file

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#626: FILE: drivers/net/i40e/base/virtchnl.h:2:
+/*******************************************************************************
+

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects?
#784: FILE: drivers/net/i40e/base/virtchnl.h:160:
+#define VIRTCHNL_CHECK_STRUCT_LEN(n, X) enum virtchnl_static_assert_enum_##X \
+	{virtchnl_static_assert_##X = (n) / ((sizeof(struct X) == (n)) ? 1 : 0)}

CHECK:MACRO_ARG_REUSE: Macro argument reuse '_v' - possible side-effects?
#825: FILE: drivers/net/i40e/base/virtchnl.h:201:
+#define VF_IS_V10(_v) (((_v)->major == 1) && ((_v)->minor == 0))

CHECK:MACRO_ARG_REUSE: Macro argument reuse '_ver' - possible side-effects?
#826: FILE: drivers/net/i40e/base/virtchnl.h:202:
+#define VF_IS_V11(_ver) (((_ver)->major == 1) && ((_ver)->minor == 1))

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#1960: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1179:
+		if (VIRTCHNL_VFR_COMPLETED == reset ||

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#1961: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1180:
+		    VIRTCHNL_VFR_VFACTIVE == reset)

total: 0 errors, 3 warnings, 3 checks, 2248 lines checked