From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8AF8DA00BE for ; Fri, 12 Jun 2020 15:39:30 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id BF8721BF57; Fri, 12 Jun 2020 15:39:20 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 993101BF53; Fri, 12 Jun 2020 15:39:19 +0200 (CEST) In-Reply-To: <20200612132934.16488-10-somnath.kotur@broadcom.com> References: <20200612132934.16488-10-somnath.kotur@broadcom.com> To: test-report@dpdk.org Cc: Somnath Kotur Message-Id: <20200612133919.993101BF53@dpdk.org> Date: Fri, 12 Jun 2020 15:39:19 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw71401 [PATCH 09/50] net/bnxt: add support for Exact Match 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: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/71401 _coding style issues_ WARNING:TYPO_SPELLING: 'becuase' may be misspelled - perhaps 'because'? #2312: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:16079: + /* Module is powered down becuase of over current fault. */ WARNING:TYPO_SPELLING: 'untill' may be misspelled - perhaps 'until'? #2370: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:20981: + * A 0xFFFF will hold the bus untill this bus is released. CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #4066: FILE: drivers/net/bnxt/tf_core/tf_core.c:414: + tbl_scope_cb = tbl_scope_cb_find( CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #4098: FILE: drivers/net/bnxt/tf_core/tf_core.c:449: + tbl_scope_cb = tbl_scope_cb_find( WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop #4878: FILE: drivers/net/bnxt/tf_core/tf_ext_flow_handle.h:93: +#define TF_GET_NUM_KEY_ENTRIES_FROM_FLOW_HANDLE(flow_handle, \ + num_key_entries) \ +do { \ + num_key_entries = \ + (((flow_handle) & TF_NUM_KEY_ENTRIES_FLOW_HANDLE_MASK) >> \ + TF_NUM_KEY_ENTRIES_FLOW_HANDLE_SFT); \ +} while (0) WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop #4886: FILE: drivers/net/bnxt/tf_core/tf_ext_flow_handle.h:101: +#define TF_GET_ENTRY_NUM_FROM_FLOW_HANDLE(flow_handle, \ + entry_num) \ +do { \ + entry_num = \ + (((flow_handle) & TF_ENTRY_NUM_FLOW_HANDLE_MASK) >> \ + TF_ENTRY_NUM_FLOW_HANDLE_SFT); \ +} while (0) CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5094: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:731: + tbl_scope_cb->em_ctx_info[TF_DIR_RX].em_tables[RECORD_TABLE].num_entries + = parms->rx_num_flows_in_k * TF_KILOBYTE; CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5096: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:733: + tbl_scope_cb->em_ctx_info[TF_DIR_RX].em_tables[RECORD_TABLE].entry_size + = parms->rx_max_action_entry_sz_in_bits / 8; CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5101: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:736: + tbl_scope_cb->em_ctx_info[TF_DIR_RX].em_tables[EFC_TABLE].num_entries + = 0; CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5114: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:750: + tbl_scope_cb->em_ctx_info[TF_DIR_TX].em_tables[RECORD_TABLE].num_entries + = parms->tx_num_flows_in_k * TF_KILOBYTE; CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5116: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:752: + tbl_scope_cb->em_ctx_info[TF_DIR_TX].em_tables[RECORD_TABLE].entry_size + = parms->tx_max_action_entry_sz_in_bits / 8; CHECK:ASSIGNMENT_CONTINUATIONS: Assignment operator '=' should be on the previous line #5121: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:755: + tbl_scope_cb->em_ctx_info[TF_DIR_TX].em_tables[EFC_TABLE].num_entries + = 0; total: 0 errors, 4 warnings, 8 checks, 5239 lines checked