From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <test-report-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 2083145A9C
	for <public@inbox.dpdk.org>; Wed,  2 Oct 2024 14:28:09 +0200 (CEST)
Received: from mails.dpdk.org (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 05A7A402DA;
	Wed,  2 Oct 2024 14:28:09 +0200 (CEST)
Received: from dpdk.org (dpdk.org [92.243.24.197])
 by mails.dpdk.org (Postfix) with ESMTP id C40D74021E
 for <test-report@dpdk.org>; Wed,  2 Oct 2024 14:28:07 +0200 (CEST)
Received: by dpdk.org (Postfix, from userid 65534)
 id AA6EB12074A; Wed,  2 Oct 2024 14:28:07 +0200 (CEST)
Subject: |WARNING| pw144767 [PATCH v3 47/47] net/bnxt: tf_ulp: add stats cache
 for thor2
In-Reply-To: <20241001055833.757163-48-sriharsha.basavapatna@broadcom.com>
References: <20241001055833.757163-48-sriharsha.basavapatna@broadcom.com>
To: test-report@dpdk.org
From: checkpatch@dpdk.org
Cc: Sriharsha Basavapatna <sriharsha.basavapatna@broadcom.com>
Message-Id: <20241002122807.AA6EB12074A@dpdk.org>
Date: Wed,  2 Oct 2024 14:28:07 +0200 (CEST)
X-BeenThere: test-report@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: automatic DPDK test reports <test-report.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
Errors-To: test-report-bounces@dpdk.org

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

_coding style issues_


WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#322: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2954:
+ulp_mapper_stats_cache_tbl_process(struct bnxt_ulp_mapper_parms *parms,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#322: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2954:
+ulp_mapper_stats_cache_tbl_process(struct bnxt_ulp_mapper_parms *parms,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#325: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2957:
+	struct ulp_flow_db_res_params fid_parms;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#340: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2972:
+	rc = ulp_mapper_tbl_result_build(parms, tbl, &data,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#351: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2983:
+	memset(&fid_parms, 0, sizeof(fid_parms));

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#351: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2983:
+	memset(&fid_parms, 0, sizeof(fid_parms));

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#352: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2984:
+	fid_parms.direction	= tbl->direction;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#353: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2985:
+	fid_parms.resource_func	= tbl->resource_func;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#354: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2986:
+	fid_parms.resource_type	= tbl->resource_type;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#355: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2987:
+	fid_parms.resource_sub_type = tbl->resource_sub_type;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#356: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2988:
+	fid_parms.resource_hndl	    = counter_handle;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#357: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2989:
+	fid_parms.critical_resource = tbl->critical_resource;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#358: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2990:
+	rc = ulp_mapper_fdb_opc_process(parms, tbl, &fid_parms);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#358: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2990:
+	rc = ulp_mapper_fdb_opc_process(parms, tbl, &fid_parms);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#365: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:2997:
+	rc = ulp_sc_mgr_entry_alloc(parms, counter_handle, tbl);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#373: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:3005:
+	BNXT_DRV_DBG(DEBUG, "flow id =0x%x
", parms->flow_id);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#395: FILE: drivers/net/bnxt/tf_ulp/ulp_mapper.c:4218:
+			rc = ulp_mapper_stats_cache_tbl_process(parms, tbl);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#489: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:73:
+		BNXT_DRV_DBG(DEBUG, "Failed to device parms
");

WARNING:LONG_LINE: line length of 109 exceeds 100 columns
#750: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:334:
+				(struct ulp_sc_tfc_stats_cache_entry *)((uintptr_t)batch_info.em_hdl[batch]);

CHECK:CAMELCASE: Avoid CamelCase: <PRIu64>
#762: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:346:
+			PMD_DRV_LOG(ERR, "%s: Stats collection time exceeded %dmS Cycles:%" PRIu64 "
",

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#888: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:472:
+int ulp_sc_mgr_entry_alloc(struct bnxt_ulp_mapper_parms *parms,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#888: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:472:
+int ulp_sc_mgr_entry_alloc(struct bnxt_ulp_mapper_parms *parms,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#895: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:479:
+	ulp_sc_info = bnxt_ulp_cntxt_ptr2_sc_info_get(parms->ulp_ctx);

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#902: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:486:
+	sce += parms->flow_id;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#911: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:495:
+	sce->ctxt = parms->ulp_ctx;

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#1072: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.h:121:
+ * parms [in] Various fields used to identify the flow

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#1077: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.h:126:
+int ulp_sc_mgr_entry_alloc(struct bnxt_ulp_mapper_parms *parms,

WARNING:TYPO_SPELLING: 'parms' may be misspelled - perhaps 'params'?
#1077: FILE: drivers/net/bnxt/tf_ulp/ulp_sc_mgr.h:126:
+int ulp_sc_mgr_entry_alloc(struct bnxt_ulp_mapper_parms *parms,

total: 0 errors, 27 warnings, 1 checks, 974 lines checked
Warning in drivers/net/bnxt/tf_ulp/ulp_sc_mgr.c:
Using pthread functions, prefer rte_thread