[dpdk-test-report] |WARNING| pw33142 [PATCH v5 01/14] net/avf/base: add base code for avf PMD

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Jan 8 22:28:00 CET 2018


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

_coding style issues_


WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE:
#64: FILE: MAINTAINERS:430:
+M: Jingjing Wu <jingjing.wu at intel.com>

WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE:
#65: FILE: MAINTAINERS:431:
+M: Wenzhuo Lu <wenzhuo.lu at intel.com>

WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE:
#66: FILE: MAINTAINERS:432:
+F: drivers/net/avf/

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#132: FILE: drivers/net/avf/base/avf_adminq.c:2:
+/*******************************************************************************
+

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.num_asq_entries == 0'
#507: FILE: drivers/net/avf/base/avf_adminq.c:377:
+	if ((hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.asq_buf_size == 0'
#507: FILE: drivers/net/avf/base/avf_adminq.c:377:
+	if ((hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.num_arq_entries == 0'
#566: FILE: drivers/net/avf/base/avf_adminq.c:436:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.arq_buf_size == 0'
#566: FILE: drivers/net/avf/base/avf_adminq.c:436:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.num_arq_entries == 0'
#685: FILE: drivers/net/avf/base/avf_adminq.c:555:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.num_asq_entries == 0'
#685: FILE: drivers/net/avf/base/avf_adminq.c:555:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.arq_buf_size == 0'
#685: FILE: drivers/net/avf/base/avf_adminq.c:555:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'hw->aq.asq_buf_size == 0'
#685: FILE: drivers/net/avf/base/avf_adminq.c:555:
+	if ((hw->aq.num_arq_entries == 0) ||
+	    (hw->aq.num_asq_entries == 0) ||
+	    (hw->aq.arq_buf_size == 0) ||
+	    (hw->aq.asq_buf_size == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around hw->aq.asq
#756: FILE: drivers/net/avf/base/avf_adminq.c:626:
+	struct avf_adminq_ring *asq = &(hw->aq.asq);

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#803: FILE: drivers/net/avf/base/avf_adminq.c:673:
+
+}

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around hw->aq.asq.r.asq_bi[hw->aq.asq.next_to_use]
#917: FILE: drivers/net/avf/base/avf_adminq.c:787:
+		dma_buff = &(hw->aq.asq.r.asq_bi[hw->aq.asq.next_to_use]);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around '!cmd_completed'
#994: FILE: drivers/net/avf/base/avf_adminq.c:864:
+	if ((!cmd_completed) &&
+	    (!details->async && !details->postpone)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'e->msg_len != 0'
#1099: FILE: drivers/net/avf/base/avf_adminq.c:969:
+	if (e->msg_buf != NULL && (e->msg_len != 0))

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1148: FILE: drivers/net/avf/base/avf_adminq.h:2:
+/*******************************************************************************
+

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#1287: FILE: drivers/net/avf/base/avf_adminq.h:141:
+		-ENOSYS,     /* AVF_AQ_RC_ENOSYS */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1320: FILE: drivers/net/avf/base/avf_adminq_cmd.h:2:
+/*******************************************************************************
+

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects?
#1659: FILE: drivers/net/avf/base/avf_adminq_cmd.h:341:
+#define AVF_CHECK_STRUCT_LEN(n, X) enum avf_static_assert_enum_##X \
+	{ avf_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }

CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
#1660: FILE: drivers/net/avf/base/avf_adminq_cmd.h:342:
+	{ avf_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
 	                             ^

WARNING:TYPO_SPELLING: 'capabilites' may be misspelled - perhaps 'capabilities'?
#1734: FILE: drivers/net/avf/base/avf_adminq_cmd.h:416:
+struct avf_aqc_list_capabilites {

WARNING:TYPO_SPELLING: 'capabilites' may be misspelled - perhaps 'capabilities'?
#1744: FILE: drivers/net/avf/base/avf_adminq_cmd.h:426:
+AVF_CHECK_CMD_LENGTH(avf_aqc_list_capabilites);

WARNING:TYPO_SPELLING: 'addres' may be misspelled - perhaps 'address'?
#3524: FILE: drivers/net/avf/base/avf_adminq_cmd.h:2206:
+	u8	dev_addres;

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#4168: FILE: drivers/net/avf/base/avf_alloc.h:2:
+/*******************************************************************************
+

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#4239: FILE: drivers/net/avf/base/avf_common.c:2:
+/*******************************************************************************
+

WARNING:EMBEDDED_FUNCTION_NAME: Prefer using '"%s...", __func__' to using 'avf_set_mac_type', this function's name, in a string
#4288: FILE: drivers/net/avf/base/avf_common.c:51:
+	DEBUGFUNC("avf_set_mac_type
");

WARNING:EMBEDDED_FUNCTION_NAME: Prefer using '"%s...", __func__' to using 'avf_set_mac_type', this function's name, in a string
#4306: FILE: drivers/net/avf/base/avf_common.c:69:
+	DEBUGOUT2("avf_set_mac_type found mac: %d, returns: %d
",

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'desc == NULL'
#4537: FILE: drivers/net/avf/base/avf_common.c:300:
+	if ((!(mask & hw->debug_mask)) || (desc == NULL))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'buffer != NULL'
#4558: FILE: drivers/net/avf/base/avf_common.c:321:
+	if ((buffer != NULL) && (aq_desc->datalen != 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'aq_desc->datalen != 0'
#4558: FILE: drivers/net/avf/base/avf_common.c:321:
+	if ((buffer != NULL) && (aq_desc->datalen != 0)) {

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4566: FILE: drivers/net/avf/base/avf_common.c:329:
+				   i, buf[i], buf[i+1], buf[i+2], buf[i+3],
 				                   ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4566: FILE: drivers/net/avf/base/avf_common.c:329:
+				   i, buf[i], buf[i+1], buf[i+2], buf[i+3],
 				                             ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4566: FILE: drivers/net/avf/base/avf_common.c:329:
+				   i, buf[i], buf[i+1], buf[i+2], buf[i+3],
 				                                       ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4567: FILE: drivers/net/avf/base/avf_common.c:330:
+				   buf[i+4], buf[i+5], buf[i+6], buf[i+7],
 				        ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4567: FILE: drivers/net/avf/base/avf_common.c:330:
+				   buf[i+4], buf[i+5], buf[i+6], buf[i+7],
 				                  ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4567: FILE: drivers/net/avf/base/avf_common.c:330:
+				   buf[i+4], buf[i+5], buf[i+6], buf[i+7],
 				                            ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4567: FILE: drivers/net/avf/base/avf_common.c:330:
+				   buf[i+4], buf[i+5], buf[i+6], buf[i+7],
 				                                      ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4568: FILE: drivers/net/avf/base/avf_common.c:331:
+				   buf[i+8], buf[i+9], buf[i+10], buf[i+11],
 				        ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4568: FILE: drivers/net/avf/base/avf_common.c:331:
+				   buf[i+8], buf[i+9], buf[i+10], buf[i+11],
 				                  ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4568: FILE: drivers/net/avf/base/avf_common.c:331:
+				   buf[i+8], buf[i+9], buf[i+10], buf[i+11],
 				                            ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4568: FILE: drivers/net/avf/base/avf_common.c:331:
+				   buf[i+8], buf[i+9], buf[i+10], buf[i+11],
 				                                       ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4569: FILE: drivers/net/avf/base/avf_common.c:332:
+				   buf[i+12], buf[i+13], buf[i+14], buf[i+15]);
 				        ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4569: FILE: drivers/net/avf/base/avf_common.c:332:
+				   buf[i+12], buf[i+13], buf[i+14], buf[i+15]);
 				                   ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4569: FILE: drivers/net/avf/base/avf_common.c:332:
+				   buf[i+12], buf[i+13], buf[i+14], buf[i+15]);
 				                              ^

CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
#4569: FILE: drivers/net/avf/base/avf_common.c:332:
+				   buf[i+12], buf[i+13], buf[i+14], buf[i+15]);
 				                                         ^

WARNING:TYPO_SPELLING: 'addres' may be misspelled - perhaps 'address'?
#5334: FILE: drivers/net/avf/base/avf_common.c:1097:
+	cmd->dev_addres = dev_addr;

WARNING:TYPO_SPELLING: 'addres' may be misspelled - perhaps 'address'?
#5368: FILE: drivers/net/avf/base/avf_common.c:1131:
+	cmd->dev_addres = dev_addr;

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#5644: FILE: drivers/net/avf/base/avf_common.c:1407:
+/**
+* avf_aq_clear_all_wol_filters

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'i == dev_cnt'
#5898: FILE: drivers/net/avf/base/avf_common.c:1661:
+	if (dev_cnt && (i == dev_cnt)) {

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6090: FILE: drivers/net/avf/base/avf_devids.h:2:
+/*******************************************************************************
+

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6139: FILE: drivers/net/avf/base/avf_hmc.h:2:
+/*******************************************************************************
+

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hw' - possible side-effects?
#6266: FILE: drivers/net/avf/base/avf_hmc.h:129:
+#define AVF_SET_PF_SD_ENTRY(hw, pa, sd_index, type)			\
+{									\
+	u32 val1, val2, val3;						\
+	val1 = (u32)(AVF_HI_DWORD(pa));				\
+	val2 = (u32)(pa) | (AVF_HMC_MAX_BP_COUNT <<			\
+		 AVF_PFHMC_SDDATALOW_PMSDBPCOUNT_SHIFT) |		\
+		((((type) == AVF_SD_TYPE_PAGED) ? 0 : 1) <<		\
+		AVF_PFHMC_SDDATALOW_PMSDTYPE_SHIFT) |			\
+		BIT(AVF_PFHMC_SDDATALOW_PMSDVALID_SHIFT);		\
+	val3 = (sd_index) | BIT_ULL(AVF_PFHMC_SDCMD_PMSDWR_SHIFT);	\
+	wr32((hw), AVF_PFHMC_SDDATAHIGH, val1);			\
+	wr32((hw), AVF_PFHMC_SDDATALOW, val2);				\
+	wr32((hw), AVF_PFHMC_SDCMD, val3);				\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pa' - possible side-effects?
#6266: FILE: drivers/net/avf/base/avf_hmc.h:129:
+#define AVF_SET_PF_SD_ENTRY(hw, pa, sd_index, type)			\
+{									\
+	u32 val1, val2, val3;						\
+	val1 = (u32)(AVF_HI_DWORD(pa));				\
+	val2 = (u32)(pa) | (AVF_HMC_MAX_BP_COUNT <<			\
+		 AVF_PFHMC_SDDATALOW_PMSDBPCOUNT_SHIFT) |		\
+		((((type) == AVF_SD_TYPE_PAGED) ? 0 : 1) <<		\
+		AVF_PFHMC_SDDATALOW_PMSDTYPE_SHIFT) |			\
+		BIT(AVF_PFHMC_SDDATALOW_PMSDVALID_SHIFT);		\
+	val3 = (sd_index) | BIT_ULL(AVF_PFHMC_SDCMD_PMSDWR_SHIFT);	\
+	wr32((hw), AVF_PFHMC_SDDATAHIGH, val1);			\
+	wr32((hw), AVF_PFHMC_SDDATALOW, val2);				\
+	wr32((hw), AVF_PFHMC_SDCMD, val3);				\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hw' - possible side-effects?
#6287: FILE: drivers/net/avf/base/avf_hmc.h:150:
+#define AVF_CLEAR_PF_SD_ENTRY(hw, sd_index, type)			\
+{									\
+	u32 val2, val3;							\
+	val2 = (AVF_HMC_MAX_BP_COUNT <<				\
+		AVF_PFHMC_SDDATALOW_PMSDBPCOUNT_SHIFT) |		\
+		((((type) == AVF_SD_TYPE_PAGED) ? 0 : 1) <<		\
+		AVF_PFHMC_SDDATALOW_PMSDTYPE_SHIFT);			\
+	val3 = (sd_index) | BIT_ULL(AVF_PFHMC_SDCMD_PMSDWR_SHIFT);	\
+	wr32((hw), AVF_PFHMC_SDDATAHIGH, 0);				\
+	wr32((hw), AVF_PFHMC_SDDATALOW, val2);				\
+	wr32((hw), AVF_PFHMC_SDCMD, val3);				\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hmc_info' - possible side-effects?
#6323: FILE: drivers/net/avf/base/avf_hmc.h:186:
+#define AVF_FIND_SD_INDEX_LIMIT(hmc_info, type, index, cnt, sd_idx, sd_limit)\
+{									\
+	u64 fpm_addr, fpm_limit;					\
+	fpm_addr = (hmc_info)->hmc_obj[(type)].base +			\
+		   (hmc_info)->hmc_obj[(type)].size * (index);		\
+	fpm_limit = fpm_addr + (hmc_info)->hmc_obj[(type)].size * (cnt);\
+	*(sd_idx) = (u32)(fpm_addr / AVF_HMC_DIRECT_BP_SIZE);		\
+	*(sd_limit) = (u32)((fpm_limit - 1) / AVF_HMC_DIRECT_BP_SIZE);	\
+	/* add one more to the limit to correct our range */		\
+	*(sd_limit) += 1;						\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'sd_limit' - possible side-effects?
#6323: FILE: drivers/net/avf/base/avf_hmc.h:186:
+#define AVF_FIND_SD_INDEX_LIMIT(hmc_info, type, index, cnt, sd_idx, sd_limit)\
+{									\
+	u64 fpm_addr, fpm_limit;					\
+	fpm_addr = (hmc_info)->hmc_obj[(type)].base +			\
+		   (hmc_info)->hmc_obj[(type)].size * (index);		\
+	fpm_limit = fpm_addr + (hmc_info)->hmc_obj[(type)].size * (cnt);\
+	*(sd_idx) = (u32)(fpm_addr / AVF_HMC_DIRECT_BP_SIZE);		\
+	*(sd_limit) = (u32)((fpm_limit - 1) / AVF_HMC_DIRECT_BP_SIZE);	\
+	/* add one more to the limit to correct our range */		\
+	*(sd_limit) += 1;						\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hmc_info' - possible side-effects?
#6347: FILE: drivers/net/avf/base/avf_hmc.h:210:
+#define AVF_FIND_PD_INDEX_LIMIT(hmc_info, type, idx, cnt, pd_index, pd_limit)\
+{									\
+	u64 fpm_adr, fpm_limit;						\
+	fpm_adr = (hmc_info)->hmc_obj[(type)].base +			\
+		  (hmc_info)->hmc_obj[(type)].size * (idx);		\
+	fpm_limit = fpm_adr + (hmc_info)->hmc_obj[(type)].size * (cnt);	\
+	*(pd_index) = (u32)(fpm_adr / AVF_HMC_PAGED_BP_SIZE);		\
+	*(pd_limit) = (u32)((fpm_limit - 1) / AVF_HMC_PAGED_BP_SIZE);	\
+	/* add one more to the limit to correct our range */		\
+	*(pd_limit) += 1;						\
+}

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pd_limit' - possible side-effects?
#6347: FILE: drivers/net/avf/base/avf_hmc.h:210:
+#define AVF_FIND_PD_INDEX_LIMIT(hmc_info, type, idx, cnt, pd_index, pd_limit)\
+{									\
+	u64 fpm_adr, fpm_limit;						\
+	fpm_adr = (hmc_info)->hmc_obj[(type)].base +			\
+		  (hmc_info)->hmc_obj[(type)].size * (idx);		\
+	fpm_limit = fpm_adr + (hmc_info)->hmc_obj[(type)].size * (cnt);	\
+	*(pd_index) = (u32)(fpm_adr / AVF_HMC_PAGED_BP_SIZE);		\
+	*(pd_limit) = (u32)((fpm_limit - 1) / AVF_HMC_PAGED_BP_SIZE);	\
+	/* add one more to the limit to correct our range */		\
+	*(pd_limit) += 1;						\
+}

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6390: FILE: drivers/net/avf/base/avf_lan_hmc.h:2:
+/*******************************************************************************
+

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#6464: FILE: drivers/net/avf/base/avf_lan_hmc.h:76:
+/* Tx queue context data
+*

ERROR:SPACING: space required after that ',' (ctx:VxV)
#6663: FILE: drivers/net/avf/base/avf_osdep.h:69:
+#define min(a,b) RTE_MIN(a,b)
              ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#6663: FILE: drivers/net/avf/base/avf_osdep.h:69:
+#define min(a,b) RTE_MIN(a,b)
                           ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#6664: FILE: drivers/net/avf/base/avf_osdep.h:70:
+#define max(a,b) RTE_MAX(a,b)
              ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#6664: FILE: drivers/net/avf/base/avf_osdep.h:70:
+#define max(a,b) RTE_MAX(a,b)
                           ^

ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
#6666: FILE: drivers/net/avf/base/avf_osdep.h:72:
+#define FIELD_SIZEOF(t, f) (sizeof(((t*)0)->f))

ERROR:SPACING: space required before the open parenthesis '('
#6667: FILE: drivers/net/avf/base/avf_osdep.h:73:
+#define ASSERT(x) if(!(x)) rte_panic("AVF: x")

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#6667: FILE: drivers/net/avf/base/avf_osdep.h:73:
+#define ASSERT(x) if(!(x)) rte_panic("AVF: x")

CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#6691: FILE: drivers/net/avf/base/avf_osdep.h:97:
+#define avf_msec_delay(x) rte_delay_us(1000*(x))
                                            ^

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(arr)
#6718: FILE: drivers/net/avf/base/avf_osdep.h:124:
+#define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))

CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
#6718: FILE: drivers/net/avf/base/avf_osdep.h:124:
+#define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))
                                     ^

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'arr' - possible side-effects?
#6718: FILE: drivers/net/avf/base/avf_osdep.h:124:
+#define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'h' - possible side-effects?
#6720: FILE: drivers/net/avf/base/avf_osdep.h:126:
+#define avf_debug(h, m, s, ...)                                \
+do {                                                            \
+	if (((m) & (h)->debug_mask))                            \
+		PMD_DRV_LOG_RAW(DEBUG, "avf %02x.%x " s,       \
+			(h)->bus.device, (h)->bus.func,         \
+					##__VA_ARGS__);         \
+} while (0)

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6766: FILE: drivers/net/avf/base/avf_prototype.h:2:
+/*******************************************************************************
+

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6978: FILE: drivers/net/avf/base/avf_register.h:2:
+/*******************************************************************************
+

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7014: FILE: drivers/net/avf/base/avf_register.h:38:
+#define AVFMSIX_PBA1(_i)          (0x00002000 + ((_i) * 4)) /* _i=0...19 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7018: FILE: drivers/net/avf/base/avf_register.h:42:
+#define AVFMSIX_TADD1(_i)              (0x00002100 + ((_i) * 16)) /* _i=0...639 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7024: FILE: drivers/net/avf/base/avf_register.h:48:
+#define AVFMSIX_TMSG1(_i)            (0x00002108 + ((_i) * 16)) /* _i=0...639 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7028: FILE: drivers/net/avf/base/avf_register.h:52:
+#define AVFMSIX_TUADD1(_i)             (0x00002104 + ((_i) * 16)) /* _i=0...639 */ /* Reset: VFLR */

WARNING:LONG_LINE: line over 90 characters
#7031: FILE: drivers/net/avf/base/avf_register.h:55:
+#define AVFMSIX_TUADD1_MSIXTUADD_MASK  AVF_MASK(0xFFFFFFFF, AVFMSIX_TUADD1_MSIXTUADD_SHIFT)

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7032: FILE: drivers/net/avf/base/avf_register.h:56:
+#define AVFMSIX_TVCTRL1(_i)        (0x0000210C + ((_i) * 16)) /* _i=0...639 */ /* Reset: VFLR */

WARNING:LONG_LINE: line over 90 characters
#7087: FILE: drivers/net/avf/base/avf_register.h:111:
+#define AVFINT_DYN_CTL01_INTENA_MASK           AVF_MASK(0x1, AVFINT_DYN_CTL01_INTENA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7089: FILE: drivers/net/avf/base/avf_register.h:113:
+#define AVFINT_DYN_CTL01_CLEARPBA_MASK         AVF_MASK(0x1, AVFINT_DYN_CTL01_CLEARPBA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7091: FILE: drivers/net/avf/base/avf_register.h:115:
+#define AVFINT_DYN_CTL01_SWINT_TRIG_MASK       AVF_MASK(0x1, AVFINT_DYN_CTL01_SWINT_TRIG_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7093: FILE: drivers/net/avf/base/avf_register.h:117:
+#define AVFINT_DYN_CTL01_ITR_INDX_MASK         AVF_MASK(0x3, AVFINT_DYN_CTL01_ITR_INDX_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7095: FILE: drivers/net/avf/base/avf_register.h:119:
+#define AVFINT_DYN_CTL01_INTERVAL_MASK         AVF_MASK(0xFFF, AVFINT_DYN_CTL01_INTERVAL_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7097: FILE: drivers/net/avf/base/avf_register.h:121:
+#define AVFINT_DYN_CTL01_SW_ITR_INDX_ENA_MASK  AVF_MASK(0x1, AVFINT_DYN_CTL01_SW_ITR_INDX_ENA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7099: FILE: drivers/net/avf/base/avf_register.h:123:
+#define AVFINT_DYN_CTL01_SW_ITR_INDX_MASK      AVF_MASK(0x3, AVFINT_DYN_CTL01_SW_ITR_INDX_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7101: FILE: drivers/net/avf/base/avf_register.h:125:
+#define AVFINT_DYN_CTL01_INTENA_MSK_MASK       AVF_MASK(0x1, AVFINT_DYN_CTL01_INTENA_MSK_SHIFT)

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7102: FILE: drivers/net/avf/base/avf_register.h:126:
+#define AVFINT_DYN_CTLN1(_INTVF)               (0x00003800 + ((_INTVF) * 4)) /* _i=0...15 */ /* Reset: VFR */

WARNING:LONG_LINE: line over 90 characters
#7105: FILE: drivers/net/avf/base/avf_register.h:129:
+#define AVFINT_DYN_CTLN1_INTENA_MASK           AVF_MASK(0x1, AVFINT_DYN_CTLN1_INTENA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7107: FILE: drivers/net/avf/base/avf_register.h:131:
+#define AVFINT_DYN_CTLN1_CLEARPBA_MASK         AVF_MASK(0x1, AVFINT_DYN_CTLN1_CLEARPBA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7109: FILE: drivers/net/avf/base/avf_register.h:133:
+#define AVFINT_DYN_CTLN1_SWINT_TRIG_MASK       AVF_MASK(0x1, AVFINT_DYN_CTLN1_SWINT_TRIG_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7111: FILE: drivers/net/avf/base/avf_register.h:135:
+#define AVFINT_DYN_CTLN1_ITR_INDX_MASK         AVF_MASK(0x3, AVFINT_DYN_CTLN1_ITR_INDX_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7113: FILE: drivers/net/avf/base/avf_register.h:137:
+#define AVFINT_DYN_CTLN1_INTERVAL_MASK         AVF_MASK(0xFFF, AVFINT_DYN_CTLN1_INTERVAL_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7115: FILE: drivers/net/avf/base/avf_register.h:139:
+#define AVFINT_DYN_CTLN1_SW_ITR_INDX_ENA_MASK  AVF_MASK(0x1, AVFINT_DYN_CTLN1_SW_ITR_INDX_ENA_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7117: FILE: drivers/net/avf/base/avf_register.h:141:
+#define AVFINT_DYN_CTLN1_SW_ITR_INDX_MASK      AVF_MASK(0x3, AVFINT_DYN_CTLN1_SW_ITR_INDX_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7119: FILE: drivers/net/avf/base/avf_register.h:143:
+#define AVFINT_DYN_CTLN1_INTENA_MSK_MASK       AVF_MASK(0x1, AVFINT_DYN_CTLN1_INTENA_MSK_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7122: FILE: drivers/net/avf/base/avf_register.h:146:
+#define AVFINT_ICR0_ENA1_LINK_STAT_CHANGE_MASK  AVF_MASK(0x1, AVFINT_ICR0_ENA1_LINK_STAT_CHANGE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7124: FILE: drivers/net/avf/base/avf_register.h:148:
+#define AVFINT_ICR0_ENA1_ADMINQ_MASK            AVF_MASK(0x1, AVFINT_ICR0_ENA1_ADMINQ_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7139: FILE: drivers/net/avf/base/avf_register.h:163:
+#define AVFINT_ICR01_LINK_STAT_CHANGE_MASK  AVF_MASK(0x1, AVFINT_ICR01_LINK_STAT_CHANGE_SHIFT)

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7144: FILE: drivers/net/avf/base/avf_register.h:168:
+#define AVFINT_ITR01(_i)            (0x00004C00 + ((_i) * 4)) /* _i=0...2 */ /* Reset: VFR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7148: FILE: drivers/net/avf/base/avf_register.h:172:
+#define AVFINT_ITRN1(_i, _INTVF)     (0x00002800 + ((_i) * 64 + (_INTVF) * 4)) /* _i=0...2, _INTVF=0...15 */ /* Reset: VFR */

WARNING:LONG_LINE: line over 90 characters
#7154: FILE: drivers/net/avf/base/avf_register.h:178:
+#define AVFINT_STAT_CTL01_OTHER_ITR_INDX_MASK  AVF_MASK(0x3, AVFINT_STAT_CTL01_OTHER_ITR_INDX_SHIFT)

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7155: FILE: drivers/net/avf/base/avf_register.h:179:
+#define AVF_QRX_TAIL1(_Q)        (0x00002000 + ((_Q) * 4)) /* _i=0...15 */ /* Reset: CORER */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7159: FILE: drivers/net/avf/base/avf_register.h:183:
+#define AVF_QTX_TAIL1(_Q)        (0x00000000 + ((_Q) * 4)) /* _i=0...15 */ /* Reset: PFR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7166: FILE: drivers/net/avf/base/avf_register.h:190:
+#define AVFMSIX_TADD(_i)              (0x00000000 + ((_i) * 16)) /* _i=0...16 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7172: FILE: drivers/net/avf/base/avf_register.h:196:
+#define AVFMSIX_TMSG(_i)            (0x00000008 + ((_i) * 16)) /* _i=0...16 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7176: FILE: drivers/net/avf/base/avf_register.h:200:
+#define AVFMSIX_TUADD(_i)             (0x00000004 + ((_i) * 16)) /* _i=0...16 */ /* Reset: VFLR */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7180: FILE: drivers/net/avf/base/avf_register.h:204:
+#define AVFMSIX_TVCTRL(_i)        (0x0000000C + ((_i) * 16)) /* _i=0...16 */ /* Reset: VFLR */

WARNING:LONG_LINE: line over 90 characters
#7193: FILE: drivers/net/avf/base/avf_register.h:217:
+#define AVFCM_PE_ERRINFO_ERROR_VALID_MASK    AVF_MASK(0x1, AVFCM_PE_ERRINFO_ERROR_VALID_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7195: FILE: drivers/net/avf/base/avf_register.h:219:
+#define AVFCM_PE_ERRINFO_ERROR_INST_MASK     AVF_MASK(0x7, AVFCM_PE_ERRINFO_ERROR_INST_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7197: FILE: drivers/net/avf/base/avf_register.h:221:
+#define AVFCM_PE_ERRINFO_DBL_ERROR_CNT_MASK  AVF_MASK(0xFF, AVFCM_PE_ERRINFO_DBL_ERROR_CNT_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7199: FILE: drivers/net/avf/base/avf_register.h:223:
+#define AVFCM_PE_ERRINFO_RLU_ERROR_CNT_MASK  AVF_MASK(0xFF, AVFCM_PE_ERRINFO_RLU_ERROR_CNT_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7201: FILE: drivers/net/avf/base/avf_register.h:225:
+#define AVFCM_PE_ERRINFO_RLS_ERROR_CNT_MASK  AVF_MASK(0xFF, AVFCM_PE_ERRINFO_RLS_ERROR_CNT_SHIFT)

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7202: FILE: drivers/net/avf/base/avf_register.h:226:
+#define AVFQF_HENA(_i)             (0x0000C400 + ((_i) * 4)) /* _i=0...1 */ /* Reset: CORER */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7206: FILE: drivers/net/avf/base/avf_register.h:230:
+#define AVFQF_HKEY(_i)         (0x0000CC00 + ((_i) * 4)) /* _i=0...12 */ /* Reset: CORER */

WARNING:LONG_LINE_COMMENT: line over 90 characters
#7226: FILE: drivers/net/avf/base/avf_register.h:250:
+#define AVFQF_HREGION(_i)                  (0x0000D400 + ((_i) * 4)) /* _i=0...7 */ /* Reset: CORER */

WARNING:LONG_LINE: line over 90 characters
#7229: FILE: drivers/net/avf/base/avf_register.h:253:
+#define AVFQF_HREGION_OVERRIDE_ENA_0_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_0_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7233: FILE: drivers/net/avf/base/avf_register.h:257:
+#define AVFQF_HREGION_OVERRIDE_ENA_1_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_1_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7237: FILE: drivers/net/avf/base/avf_register.h:261:
+#define AVFQF_HREGION_OVERRIDE_ENA_2_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_2_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7241: FILE: drivers/net/avf/base/avf_register.h:265:
+#define AVFQF_HREGION_OVERRIDE_ENA_3_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_3_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7245: FILE: drivers/net/avf/base/avf_register.h:269:
+#define AVFQF_HREGION_OVERRIDE_ENA_4_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_4_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7249: FILE: drivers/net/avf/base/avf_register.h:273:
+#define AVFQF_HREGION_OVERRIDE_ENA_5_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_5_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7253: FILE: drivers/net/avf/base/avf_register.h:277:
+#define AVFQF_HREGION_OVERRIDE_ENA_6_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_6_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7257: FILE: drivers/net/avf/base/avf_register.h:281:
+#define AVFQF_HREGION_OVERRIDE_ENA_7_MASK  AVF_MASK(0x1, AVFQF_HREGION_OVERRIDE_ENA_7_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7262: FILE: drivers/net/avf/base/avf_register.h:286:
+#define AVFINT_DYN_CTL01_WB_ON_ITR_MASK        AVF_MASK(0x1, AVFINT_DYN_CTL01_WB_ON_ITR_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7264: FILE: drivers/net/avf/base/avf_register.h:288:
+#define AVFINT_DYN_CTLN1_WB_ON_ITR_MASK        AVF_MASK(0x1, AVFINT_DYN_CTLN1_WB_ON_ITR_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7270: FILE: drivers/net/avf/base/avf_register.h:294:
+#define AVFPE_CCQPHIGH1_PECCQPHIGH_MASK  AVF_MASK(0xFFFFFFFF, AVFPE_CCQPHIGH1_PECCQPHIGH_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7273: FILE: drivers/net/avf/base/avf_register.h:297:
+#define AVFPE_CCQPLOW1_PECCQPLOW_MASK  AVF_MASK(0xFFFFFFFF, AVFPE_CCQPLOW1_PECCQPLOW_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7276: FILE: drivers/net/avf/base/avf_register.h:300:
+#define AVFPE_CCQPSTATUS1_CCQP_DONE_MASK    AVF_MASK(0x1, AVFPE_CCQPSTATUS1_CCQP_DONE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7278: FILE: drivers/net/avf/base/avf_register.h:302:
+#define AVFPE_CCQPSTATUS1_HMC_PROFILE_MASK  AVF_MASK(0x7, AVFPE_CCQPSTATUS1_HMC_PROFILE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7280: FILE: drivers/net/avf/base/avf_register.h:304:
+#define AVFPE_CCQPSTATUS1_RDMA_EN_VFS_MASK  AVF_MASK(0x3F, AVFPE_CCQPSTATUS1_RDMA_EN_VFS_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7282: FILE: drivers/net/avf/base/avf_register.h:306:
+#define AVFPE_CCQPSTATUS1_CCQP_ERR_MASK     AVF_MASK(0x1, AVFPE_CCQPSTATUS1_CCQP_ERR_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7294: FILE: drivers/net/avf/base/avf_register.h:318:
+#define AVFPE_CQPERRCODES1_CQP_MINOR_CODE_MASK  AVF_MASK(0xFFFF, AVFPE_CQPERRCODES1_CQP_MINOR_CODE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7296: FILE: drivers/net/avf/base/avf_register.h:320:
+#define AVFPE_CQPERRCODES1_CQP_MAJOR_CODE_MASK  AVF_MASK(0xFFFF, AVFPE_CQPERRCODES1_CQP_MAJOR_CODE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7304: FILE: drivers/net/avf/base/avf_register.h:328:
+#define AVFPE_IPCONFIG01_PEIPID_MASK            AVF_MASK(0xFFFF, AVFPE_IPCONFIG01_PEIPID_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7306: FILE: drivers/net/avf/base/avf_register.h:330:
+#define AVFPE_IPCONFIG01_USEENTIREIDRANGE_MASK  AVF_MASK(0x1, AVFPE_IPCONFIG01_USEENTIREIDRANGE_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7309: FILE: drivers/net/avf/base/avf_register.h:333:
+#define AVFPE_MRTEIDXMASK1_MRTEIDXMASKBITS_MASK  AVF_MASK(0x1F, AVFPE_MRTEIDXMASK1_MRTEIDXMASKBITS_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7312: FILE: drivers/net/avf/base/avf_register.h:336:
+#define AVFPE_RCVUNEXPECTEDERROR1_TCP_RX_UNEXP_ERR_MASK  AVF_MASK(0xFFFFFF, AVFPE_RCVUNEXPECTEDERROR1_TCP_RX_UNEXP_ERR_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7315: FILE: drivers/net/avf/base/avf_register.h:339:
+#define AVFPE_TCPNOWTIMER1_TCP_NOW_MASK  AVF_MASK(0xFFFFFFFF, AVFPE_TCPNOWTIMER1_TCP_NOW_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7318: FILE: drivers/net/avf/base/avf_register.h:342:
+#define AVFPE_WQEALLOC1_PEQPID_MASK          AVF_MASK(0x3FFFF, AVFPE_WQEALLOC1_PEQPID_SHIFT)

WARNING:LONG_LINE: line over 90 characters
#7320: FILE: drivers/net/avf/base/avf_register.h:344:
+#define AVFPE_WQEALLOC1_WQE_DESC_INDEX_MASK  AVF_MASK(0xFFF, AVFPE_WQEALLOC1_WQE_DESC_INDEX_SHIFT)

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#7330: FILE: drivers/net/avf/base/avf_status.h:2:
+/*******************************************************************************
+

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#7444: FILE: drivers/net/avf/base/avf_type.h:2:
+/*******************************************************************************
+

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#7488: FILE: drivers/net/avf/base/avf_type.h:46:
+#define UNREFERENCED_1PARAMETER(_p) (_p);

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#7489: FILE: drivers/net/avf/base/avf_type.h:47:
+#define UNREFERENCED_2PARAMETER(_p, _q) (_p); (_q);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#7489: FILE: drivers/net/avf/base/avf_type.h:47:
+#define UNREFERENCED_2PARAMETER(_p, _q) (_p); (_q);

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#7490: FILE: drivers/net/avf/base/avf_type.h:48:
+#define UNREFERENCED_3PARAMETER(_p, _q, _r) (_p); (_q); (_r);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#7490: FILE: drivers/net/avf/base/avf_type.h:48:
+#define UNREFERENCED_3PARAMETER(_p, _q, _r) (_p); (_q); (_r);

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#7491: FILE: drivers/net/avf/base/avf_type.h:49:
+#define UNREFERENCED_4PARAMETER(_p, _q, _r, _s) (_p); (_q); (_r); (_s);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#7491: FILE: drivers/net/avf/base/avf_type.h:49:
+#define UNREFERENCED_4PARAMETER(_p, _q, _r, _s) (_p); (_q); (_r); (_s);

ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros with multiple statements should be enclosed in a do - while loop
#7492: FILE: drivers/net/avf/base/avf_type.h:50:
+#define UNREFERENCED_5PARAMETER(_p, _q, _r, _s, _t) (_p); (_q); (_r); (_s); (_t);

WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#7492: FILE: drivers/net/avf/base/avf_type.h:50:
+#define UNREFERENCED_5PARAMETER(_p, _q, _r, _s, _t) (_p); (_q); (_r); (_s); (_t);

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'address' - possible side-effects?
#7529: FILE: drivers/net/avf/base/avf_type.h:87:
+#define AVF_IS_BROADCAST(address)	\
+	((((u8 *)(address))[0] == ((u8)0xff)) && \
+	(((u8 *)(address))[1] == ((u8)0xff)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'R' - possible side-effects?
#7560: FILE: drivers/net/avf/base/avf_type.h:118:
+#define AVF_DESC_UNUSED(R)	\
+	((((R)->next_to_clean > (R)->next_to_use) ? 0 : (R)->count) + \
+	(R)->next_to_clean - (R)->next_to_use - 1)

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

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

total: 11 errors, 100 warnings, 50 checks, 10104 lines checked


More information about the test-report mailing list