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 751B1A2EFC for ; Tue, 15 Oct 2019 17:44:24 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 69E1E1ED29; Tue, 15 Oct 2019 17:44:24 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 90FA61ED26; Tue, 15 Oct 2019 17:44:22 +0200 (CEST) In-Reply-To: <20191014052202.12625-1-asomalap@amd.com> References: <20191014052202.12625-1-asomalap@amd.com> To: test-report@dpdk.org Cc: Message-Id: <20191015154422.90FA61ED26@dpdk.org> Date: Tue, 15 Oct 2019 17:44:22 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw61041 [PATCH v1 6/6] crypto/ccp: fix for scheduling multiple CCP within single burst ccp driver was scheduling only one CCP in a 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/61041 _coding style issues_ ERROR:SPACING: space required before the open parenthesis '(' #114: FILE: drivers/crypto/ccp/ccp_crypto.c:3079: + } else if(b_info->tail_offset != b_info->head_offset) { ERROR:GLOBAL_INITIALISERS: do not initialise globals to 0 #174: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:26: +uint8_t cryptodev_cnt = 0; ERROR:SPACING: space required after that ',' (ctx:VxV) #182: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:206: + uint16_t tmp_ops = nb_ops,b_idx,cur_ops=0; ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #182: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:206: + uint16_t tmp_ops = nb_ops,b_idx,cur_ops=0; ^ ERROR:SPACING: spaces required around that '=' (ctx:VxV) #182: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:206: + uint16_t tmp_ops = nb_ops,b_idx,cur_ops=0; ^ ERROR:OPEN_BRACE: that open brace { should be on the previous line #189: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:213: + if(tmp_ops >= cryptodev_cnt) + { ERROR:SPACING: space required before the open parenthesis '(' #189: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:213: + if(tmp_ops >= cryptodev_cnt) ERROR:SPACING: need consistent spacing around '+' (ctx:VxW) #191: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:215: + cur_ops = nb_ops / cryptodev_cnt+ (nb_ops)%cryptodev_cnt ; ^ WARNING:SPACING: space prohibited before semicolon #191: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:215: + cur_ops = nb_ops / cryptodev_cnt+ (nb_ops)%cryptodev_cnt ; ERROR:ELSE_AFTER_BRACE: else should follow close brace '}' #193: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:217: + } + else ERROR:OPEN_BRACE: that open brace { should be on the previous line #195: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:219: + while(tmp_ops) + { ERROR:SPACING: space required before the open parenthesis '(' #195: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:219: + while(tmp_ops) ERROR:OPEN_BRACE: that open brace { should be on the previous line #199: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:223: + if(cur_ops <= tmp_ops ) + { ERROR:SPACING: space prohibited before that close parenthesis ')' #199: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:223: + if(cur_ops <= tmp_ops ) ERROR:SPACING: space required before the open parenthesis '(' #199: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:223: + if(cur_ops <= tmp_ops ) ERROR:CODE_INDENT: code indent should use tabs where possible #201: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:225: +^I tmp_ops -= cur_ops;$ ERROR:OPEN_BRACE: that open brace { should be on the previous line #203: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:227: + else + { ERROR:ELSE_AFTER_BRACE: else should follow close brace '}' #203: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:227: + } + else WARNING:LONG_LINE: line over 90 characters #232: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:247: + enq_cnt += process_ops_to_enqueue(qp, ops, cmd_q, cur_ops, nb_ops, slots_req, b_idx); ERROR:SPACING: space required after that ',' (ctx:VxO) #251: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:261: + nb_dequeued = process_ops_to_dequeue(qp, ops, nb_ops,&total_nb_ops); ^ ERROR:SPACING: space required before that '&' (ctx:OxV) #251: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:261: + nb_dequeued = process_ops_to_dequeue(qp, ops, nb_ops,&total_nb_ops); ^ ERROR:OPEN_BRACE: that open brace { should be on the previous line #254: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:263: + if(total_nb_ops) + { ERROR:SPACING: space required before the open parenthesis '(' #254: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:263: + if(total_nb_ops) ERROR:OPEN_BRACE: that open brace { should be on the previous line #256: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:265: + while(nb_dequeued != total_nb_ops) + { ERROR:SPACING: space required before the open parenthesis '(' #256: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:265: + while(nb_dequeued != total_nb_ops) WARNING:LONG_LINE: line over 90 characters #258: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:267: + nb_dequeued = process_ops_to_dequeue(qp, ops, nb_ops,&total_nb_ops); ERROR:SPACING: space required after that ',' (ctx:VxO) #258: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:267: + nb_dequeued = process_ops_to_dequeue(qp, ops, nb_ops,&total_nb_ops); ^ ERROR:SPACING: space required before that '&' (ctx:OxV) #258: FILE: drivers/crypto/ccp/rte_ccp_pmd.c:267: + nb_dequeued = process_ops_to_dequeue(qp, ops, nb_ops,&total_nb_ops); ^ total: 25 errors, 3 warnings, 202 lines checked