From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 485A6461C1 for ; Sat, 8 Feb 2025 03:46:10 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 39D6D4065D; Sat, 8 Feb 2025 03:46:10 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id A8EF540395 for ; Sat, 8 Feb 2025 03:46:08 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id 9CE4B127003; Sat, 8 Feb 2025 03:44:55 +0100 (CET) Subject: |WARNING| pw151147 [PATCH v7 05/28] net/rnp: add device init and uninit In-Reply-To: <1738982645-34550-6-git-send-email-caowenbo@mucse.com> References: <1738982645-34550-6-git-send-email-caowenbo@mucse.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Wenbo Cao Message-Id: <20250208024455.9CE4B127003@dpdk.org> Date: Sat, 8 Feb 2025 03:44:55 +0100 (CET) X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 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 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/151147 _coding style issues_ WARNING:TYPO_SPELLING: 'availble' may be misspelled - perhaps 'available'? #425: FILE: drivers/net/rnp/base/rnp_fw_cmd.h:108: + u32 rpu_availble : 1; /* 11 */ WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'? #564: FILE: drivers/net/rnp/base/rnp_hw.h:73: + bool msg_alloced; CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #567: FILE: drivers/net/rnp/base/rnp_hw.h:76: + spinlock_t fw_lock; CHECK:MACRO_ARG_REUSE: Macro argument reuse 'func' - possible side-effects? #570: FILE: drivers/net/rnp/base/rnp_hw.h:79: +#define rnp_call_hwif_impl(port, func, arg...) \ + (((func) != NULL) ? ((func) (port, arg)) : (-ENODEV)) CHECK:SPACING: No space is necessary after a cast #1113: FILE: drivers/net/rnp/base/rnp_osdep.h:92: + static RTE_ATOMIC(uint64_t) rnp_dma_memzone_id; CHECK:CAMELCASE: Avoid CamelCase: #1125: FILE: drivers/net/rnp/base/rnp_osdep.h:104: + snprintf(z_name, sizeof(z_name), "rnp_dma_%" PRIu64, CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #1139: FILE: drivers/net/rnp/base/rnp_osdep.h:118: + "%"PRIu64, mz->name, mem->pa); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #1149: FILE: drivers/net/rnp/base/rnp_osdep.h:128: + "%"PRIu64, ((const struct rte_memzone *)mem->mz)->name, WARNING:TYPO_SPELLING: 'registed' may be misspelled - perhaps 'registered'? #1217: FILE: drivers/net/rnp/rnp.h:48: + bool intr_registed; CHECK:MACRO_ARG_REUSE: Macro argument reuse 'id' - possible side-effects? #1231: FILE: drivers/net/rnp/rnp.h:62: +#define RNP_PF_OWN_PORTS(id) (((id) == 0) ? 1 : (((id) == 1) ? 2 : 4)) WARNING:TYPO_SPELLING: 'registed' may be misspelled - perhaps 'registered'? #1319: FILE: drivers/net/rnp/rnp_ethdev.c:75: + if (adapter->intr_registed) { WARNING:TYPO_SPELLING: 'registed' may be misspelled - perhaps 'registered'? #1325: FILE: drivers/net/rnp/rnp_ethdev.c:81: + adapter->intr_registed = false; WARNING:TYPO_SPELLING: 'plaform' may be misspelled - perhaps 'platform'? #1515: FILE: drivers/net/rnp/rnp_ethdev.c:269: + /* port 0 will be probe by plaform */ WARNING:TYPO_SPELLING: 'registed' may be misspelled - perhaps 'registered'? #1523: FILE: drivers/net/rnp/rnp_ethdev.c:277: + adapter->intr_registed = true; WARNING:TYPO_SPELLING: 'alloced' may be misspelled - perhaps 'allocated'? #1535: FILE: drivers/net/rnp/rnp_ethdev.c:289: + /* just release eth_dev alloced by myself */ total: 0 errors, 8 warnings, 7 checks, 1423 lines checked Warning in drivers/net/rnp/base/rnp_osdep.h: Using compiler attribute directly Warning in drivers/net/rnp/base/rnp_hw.h: Do not use variadic argument pack in macros