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 0D32B4649D for ; Fri, 28 Mar 2025 06:17:40 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 01B6840652; Fri, 28 Mar 2025 06:17:40 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 9BB8C402A7 for ; Fri, 28 Mar 2025 06:17:37 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id 0D8A2123E9B; Fri, 28 Mar 2025 06:17:14 +0100 (CET) Subject: |WARNING| pw152635 [PATCH v17 04/29] net/rnp: support mailbox basic operate In-Reply-To: <20250328051444.1019208-5-caowenbo@mucse.com> References: <20250328051444.1019208-5-caowenbo@mucse.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Wenbo Cao Message-Id: <20250328051714.0D8A2123E9B@dpdk.org> Date: Fri, 28 Mar 2025 06:17:14 +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/152635 _coding style issues_ WARNING:MEMORY_BARRIER: memory barrier without comment #220: FILE: drivers/net/rnp/base/rnp_mbx.c:21: + mb(); WARNING:MEMORY_BARRIER: memory barrier without comment #235: FILE: drivers/net/rnp/base/rnp_mbx.c:36: + mb(); WARNING:MEMORY_BARRIER: memory barrier without comment #264: FILE: drivers/net/rnp/base/rnp_mbx.c:65: + mb(); WARNING:MEMORY_BARRIER: memory barrier without comment #290: FILE: drivers/net/rnp/base/rnp_mbx.c:91: + mb(); WARNING:MEMORY_BARRIER: memory barrier without comment #324: FILE: drivers/net/rnp/base/rnp_mbx.c:125: + mb(); WARNING:MEMORY_BARRIER: memory barrier without comment #431: FILE: drivers/net/rnp/base/rnp_mbx.c:232: + wmb(); CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #435: FILE: drivers/net/rnp/base/rnp_mbx.c:236: + udelay(100); WARNING:MEMORY_BARRIER: memory barrier without comment #501: FILE: drivers/net/rnp/base/rnp_mbx.c:302: + mb(); CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #544: FILE: drivers/net/rnp/base/rnp_mbx.c:345: + udelay(300); WARNING:MEMORY_BARRIER: memory barrier without comment #545: FILE: drivers/net/rnp/base/rnp_mbx.c:346: + mb(); WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'? #733: FILE: drivers/net/rnp/base/rnp_mbx.h:21: + *|------master-req-----|-------master-ack------| WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'? #733: FILE: drivers/net/rnp/base/rnp_mbx.h:21: + *|------master-req-----|-------master-ack------| WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'? #734: FILE: drivers/net/rnp/base/rnp_mbx.h:22: + *|------slave-req------|-------slave-ack-------| WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'? #734: FILE: drivers/net/rnp/base/rnp_mbx.h:22: + *|------slave-req------|-------slave-ack-------| WARNING:MEMORY_BARRIER: memory barrier without comment #799: FILE: drivers/net/rnp/base/rnp_osdep.h:22: +#define mb() rte_mb() WARNING:MEMORY_BARRIER: memory barrier without comment #800: FILE: drivers/net/rnp/base/rnp_osdep.h:23: +#define wmb() rte_wmb() CHECK:CAMELCASE: Avoid CamelCase: #812: FILE: drivers/net/rnp/base/rnp_osdep.h:35: + RNP_PMD_REG_LOG(DEBUG, "offset=%p val=%#"PRIx32"", offset, v); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #812: FILE: drivers/net/rnp/base/rnp_osdep.h:35: + RNP_PMD_REG_LOG(DEBUG, "offset=%p val=%#"PRIx32"", offset, v); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #819: FILE: drivers/net/rnp/base/rnp_osdep.h:42: + RNP_PMD_REG_LOG(DEBUG, "offset=%p val=%#"PRIx32"", offset, val); CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hw' - possible side-effects? #827: FILE: drivers/net/rnp/base/rnp_osdep.h:50: +#define RNP_C_REG_WR(hw, off, value) \ + rnp_reg_write32((hw)->c_ctrl, ((off) & ((hw)->c_blen - 1)), (value)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'hw' - possible side-effects? #829: FILE: drivers/net/rnp/base/rnp_osdep.h:52: +#define RNP_C_REG_RD(hw, off) \ + rnp_reg_read32((hw)->c_ctrl, ((off) & ((hw)->c_blen - 1))) total: 0 errors, 14 warnings, 7 checks, 752 lines checked