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 65E0B4596C for ; Thu, 12 Sep 2024 10:29:22 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5D31542709; Thu, 12 Sep 2024 10:29:22 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 324824113C for ; Thu, 12 Sep 2024 10:29:18 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 1D57C121D10; Thu, 12 Sep 2024 10:29:18 +0200 (CEST) Subject: |WARNING| pw143990 [PATCH v2 10/14] drivers: replace logging helpers In-Reply-To: <20240912082643.1532679-11-david.marchand@redhat.com> References: <20240912082643.1532679-11-david.marchand@redhat.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: David Marchand Message-Id: <20240912082918.1D57C121D10@dpdk.org> Date: Thu, 12 Sep 2024 10:29:18 +0200 (CEST) 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/143990 _coding style issues_ ERROR:SPACING: space required after that ',' (ctx:VxB) #329: FILE: drivers/bus/platform/private.h:38: + RTE_LOG(level, PLATFORM_BUS, RTE_FMT(RTE_FMT_HEAD(__VA_ARGS__,) " ", \ ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #330: FILE: drivers/bus/platform/private.h:39: + RTE_FMT_TAIL(__VA_ARGS__,))) ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #608: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:340: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #652: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:366: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #657: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:371: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #665: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:376: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #685: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:391: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #704: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:405: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #723: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:419: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #736: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:424: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #741: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:429: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #769: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:439: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #774: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:444: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #782: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:452: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #1241: FILE: drivers/dma/odm/odm.h:71: + RTE_FMT("%s(): %u" RTE_FMT_HEAD(__VA_ARGS__,) " ", __func__, __LINE__, \ ^ WARNING:TYPO_SPELLING: 'hist' may be misspelled - perhaps 'heist'? #1287: FILE: drivers/event/dlb2/dlb2.c:835: + DLB2_LOG_LINE_DBG("sched domain create - ldb_qs=%d, ldb_ports=%d, dir_ports=%d, atomic_inflights=%d, hist_list_entries=%d, credits=%d", WARNING:TYPO_SPELLING: 'hist' may be misspelled - perhaps 'heist'? #1296: FILE: drivers/event/dlb2/dlb2.c:843: + DLB2_LOG_LINE_DBG("sched domain create - ldb_qs=%d, ldb_ports=%d, dir_ports=%d, atomic_inflights=%d, hist_list_entries=%d, ldb_credits=%d, dir_credits=%d", CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3709: FILE: drivers/net/axgbe/axgbe_ethdev.c:1777: + "flags = 0x%x nsec = %"PRIu64, CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3725: FILE: drivers/net/axgbe/axgbe_ethdev.c:1810: + PMD_DRV_LOG_LINE(DEBUG, "nsec = %"PRIu64" tx_ssr = %d tx_snr = %d", WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'? #3916: FILE: drivers/net/axgbe/axgbe_i2c.c:235: + PMD_DRV_LOG_LINE(ERR, "failed to disable i2c master"); WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'? #3925: FILE: drivers/net/axgbe/axgbe_i2c.c:252: + PMD_DRV_LOG_LINE(ERR, "failed to enable i2c master"); WARNING:TYPO_SPELLING: 'master' may be misspelled - perhaps 'primary'? #3961: FILE: drivers/net/axgbe/axgbe_i2c.c:324: + PMD_DRV_LOG_LINE(ERR, "failed to disable i2c master"); CHECK:CAMELCASE: Avoid CamelCase: #10903: FILE: drivers/net/ena/ena_ethdev.c:2887: + "mbuf[%" PRIu32 "] has unsupported offloads flags set: 0x%" PRIu64, CHECK:CAMELCASE: Avoid CamelCase: #10914: FILE: drivers/net/ena/ena_ethdev.c:2898: + "mbuf[%" PRIu32 "] has too many segments: %" PRIu16, CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #11432: FILE: drivers/net/ena/ena_rss.c:427: + "Setting RSS hash fields is not supported. Using default values: 0x%"PRIx64, total: 15 errors, 5 warnings, 5 checks, 10498 lines checked