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

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jun 29 23:53:23 CEST 2017


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

_coding style issues_


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

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

WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#1789: FILE: drivers/net/i40e/i40e_ethdev_vf.c:727:
+	uint8_t cmd_buffer[sizeof(struct virtchnl_irq_map_info) + \

WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#1843: FILE: drivers/net/i40e/i40e_ethdev_vf.c:866:
+	uint8_t cmd_buffer[sizeof(struct virtchnl_ether_addr_list) + \

WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#1873: FILE: drivers/net/i40e/i40e_ethdev_vf.c:904:
+	uint8_t cmd_buffer[sizeof(struct virtchnl_ether_addr_list) + \

total: 0 errors, 4 warnings, 3 checks, 2216 lines checked


More information about the test-report mailing list