From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id C9E357D66; Fri, 25 Aug 2017 20:41:31 +0200 (CEST) In-Reply-To: <20170825184023.31692-2-yskoh@mellanox.com> References: <20170825184023.31692-2-yskoh@mellanox.com> To: test-report@dpdk.org Cc: Yongseok Koh Message-Id: <20170825184131.C9E357D66@dpdk.org> Date: Fri, 25 Aug 2017 20:41:31 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw28015 [PATCH RFC 1/1] net/mlx5: add vectorized Rx/Tx burst for ARM 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, 25 Aug 2017 18:41:31 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/28015 _coding style issues_ CHECK:REDUNDANT_CODE: if this code is redundant consider removing it #73: FILE: drivers/net/mlx5/mlx5_prm.h:227: +#if 0 WARNING:LEADING_SPACE: please, no spaces at the start of a line #75: FILE: drivers/net/mlx5/mlx5_prm.h:229: + uint16_t hdr_type_etc;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #76: FILE: drivers/net/mlx5/mlx5_prm.h:230: + uint8_t pkt_info;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #77: FILE: drivers/net/mlx5/mlx5_prm.h:231: + uint8_t sop_drop_qpn; /* flow_tag */$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #78: FILE: drivers/net/mlx5/mlx5_prm.h:232: + uint16_t byte_cnt;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #79: FILE: drivers/net/mlx5/mlx5_prm.h:233: + uint16_t vlan_info;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #80: FILE: drivers/net/mlx5/mlx5_prm.h:234: + uint32_t rx_hash_res;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #81: FILE: drivers/net/mlx5/mlx5_prm.h:235: + uint8_t timestamp;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #82: FILE: drivers/net/mlx5/mlx5_prm.h:236: + uint8_t wqe_counter;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #83: FILE: drivers/net/mlx5/mlx5_prm.h:237: + uint8_t rsvd4;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #84: FILE: drivers/net/mlx5/mlx5_prm.h:238: + uint8_t op_own;$ CHECK:REDUNDANT_CODE: if this code is redundant consider removing it #322: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:114: +#if 0 CHECK:REDUNDANT_CODE: if this code is redundant consider removing it #678: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:470: +#if 0 ERROR:SPACING: spaces required around that ':' (ctx:ExV) #928: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:720: + :[byte_cnt]"=&w"(byte_cnt) ^ ERROR:SPACING: spaces required around that ':' (ctx:ExV) #929: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:721: + :[mcq]"r"(p), [rxdf]"w"(rxdf), [rearm]"w"(rearm), ^ ERROR:SPACING: spaces required around that ':' (ctx:ExV) #934: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:726: + :"memory", "v16", "v17", "v18", "v19"); ^ CHECK:REDUNDANT_CODE: if this code is redundant consider removing it #970: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:762: +#if 0 ERROR:SPACING: spaces required around that ':' (ctx:ExV) #1417: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:1209: + :[op_own]"=&w"(op_own), [byte_cnt]"=&w"(byte_cnt) ^ ERROR:SPACING: spaces required around that ':' (ctx:ExV) #1418: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:1210: + :[p3]"r"(p3 + 48), [p2]"r"(p2 + 48), ^ ERROR:SPACING: spaces required around that ':' (ctx:ExV) #1425: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:1217: + :"memory", ^ WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines #1469: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:1261: + /* TODO: + rxq_cq_to_ptype_oflags_v(rxq, cqes, opcode, &pkts[pos]); */ WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line #1469: FILE: drivers/net/mlx5/mlx5_rxtx_vec_neon.c:1261: + rxq_cq_to_ptype_oflags_v(rxq, cqes, opcode, &pkts[pos]); */ total: 6 errors, 12 warnings, 4 checks, 1608 lines checked