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 8398DA0524
	for <public@inbox.dpdk.org>; Sun, 30 May 2021 11:03:02 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id C520440E6E;
	Sun, 30 May 2021 11:03:01 +0200 (CEST)
Received: from dpdk.org (dpdk.org [92.243.24.197])
 by mails.dpdk.org (Postfix) with ESMTP id 5FCEA40E01
 for <test-report@dpdk.org>; Sun, 30 May 2021 11:03:01 +0200 (CEST)
Received: by dpdk.org (Postfix, from userid 65534)
 id 5CD07122E1E; Sun, 30 May 2021 11:03:01 +0200 (CEST)
In-Reply-To: <20210530085929.29695-12-venkatkumar.duvvuru@broadcom.com>
References: <20210530085929.29695-12-venkatkumar.duvvuru@broadcom.com>
To: test-report@dpdk.org
From: checkpatch@dpdk.org
Cc: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>
Message-Id: <20210530090301.5CD07122E1E@dpdk.org>
Date: Sun, 30 May 2021 11:03:01 +0200 (CEST)
Subject: [dpdk-test-report] |WARNING| pw93560 [PATCH 11/58] net/bnxt: add
 hashing changes for Thor
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
Sender: "test-report" <test-report-bounces@dpdk.org>

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

_coding style issues_


WARNING:TYPO_SPELLING: 'bae' may be misspelled - perhaps 'base'?
#236: FILE: drivers/net/bnxt/hcapi/cfa/hcapi_cfa_common.c:61:
+0x95bf4a82, 0xe2b87a14, 0x7bb12bae, 0x0cb61b38,

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'crc' - possible side-effects?
#275: FILE: drivers/net/bnxt/hcapi/cfa/hcapi_cfa_defs.h:63:
+#define ucrc32(ch, crc) (crc32tbl[((crc) ^ (ch)) & 0xff] ^ ((crc) >> 8))

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#308: FILE: drivers/net/bnxt/hcapi/cfa/hcapi_cfa_p4.c:19:
+extern const uint32_t crc32tbl[];

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#450: FILE: drivers/net/bnxt/hcapi/cfa/hcapi_cfa_p58.c:19:
+extern const uint32_t crc32tbl[];

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#617: FILE: drivers/net/bnxt/tf_core/tf_device_p4.c:21:
+uint64_t hcapi_cfa_key_hash_p4(uint64_t *key_data,

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#637: FILE: drivers/net/bnxt/tf_core/tf_device_p58.c:21:
+uint64_t hcapi_cfa_key_hash_p58(uint64_t *key_data,

total: 0 errors, 5 warnings, 1 checks, 667 lines checked