From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id 246CA1B00B; Fri, 5 Jan 2018 10:20:13 +0100 (CET) In-Reply-To: <1515140505-38655-2-git-send-email-wenzhuo.lu@intel.com> References: <1515140505-38655-2-git-send-email-wenzhuo.lu@intel.com> To: test-report@dpdk.org Cc: Wenzhuo Lu Message-Id: <20180105092013.246CA1B00B@dpdk.org> Date: Fri, 5 Jan 2018 10:20:13 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw32945 [PATCH v4 01/15] net/avf/base: add base code for avf PMD X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 Jan 2018 09:20:13 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/32945 _coding style issues_ WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE: #64: FILE: MAINTAINERS:429: +M: Jingjing Wu WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE: #65: FILE: MAINTAINERS:430: +M: Wenzhuo Lu WARNING:MAINTAINERS_STYLE: MAINTAINERS entries use one tab after TYPE: #66: FILE: MAINTAINERS:431: +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