[dpdk-test-report] |WARNING| pw71408 [PATCH 16/50] net/bnxt: add core changes for EM and EEM lookups

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Jun 12 15:42:58 CEST 2020


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

_coding style issues_


WARNING:LONG_LINE: line over 90 characters
#3174: FILE: drivers/net/bnxt/tf_core/tf_em.c:119:
+		(uint8_t *)&tbl_scope_cb->em_ctx_info[parms->dir].em_tables[TF_KEY0_TABLE];

WARNING:LONG_LINE: line over 90 characters
#3190: FILE: drivers/net/bnxt/tf_core/tf_em.c:135:
+			(uint8_t *)&tbl_scope_cb->em_ctx_info[parms->dir].em_tables[TF_KEY1_TABLE];

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#3240: FILE: drivers/net/bnxt/tf_core/tf_em.c:204:
+	PMD_DRV_LOG(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '['
#3292: FILE: drivers/net/bnxt/tf_core/tf_em.c:288:
+		(uint8_t *)&tbl_scope_cb->em_ctx_info[parms->dir].em_tables[

WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#3989: FILE: drivers/net/bnxt/tf_core/tf_rm.c:3189:
+#if 0

CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#4025: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:64:
+				    "No mapping for page: %d table: %016" PRIu64 "
",

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'parms->rx_num_flows_in_k != 0'
#4247: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:706:
+	if ((parms->rx_num_flows_in_k != 0) &&
 	    (parms->rx_max_key_sz_in_bits / 8 == 0)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'parms->tx_num_flows_in_k != 0'
#4257: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:714:
+	if ((parms->tx_num_flows_in_k != 0) &&
 	    (parms->tx_max_key_sz_in_bits / 8 == 0)) {

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#4772: FILE: drivers/net/bnxt/tf_core/tf_tbl.c:1632:
+		base_addr = (void *)(uintptr_t)hcapi_get_table_page(

total: 0 errors, 3 warnings, 6 checks, 4576 lines checked


More information about the test-report mailing list