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 5C0DE4592F for ; Sat, 7 Sep 2024 16:59:26 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 58FAF4026F; Sat, 7 Sep 2024 16:59:26 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id E71E640269 for ; Sat, 7 Sep 2024 16:59:24 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id E14EF121D10; Sat, 7 Sep 2024 16:59:24 +0200 (CEST) Subject: |WARNING| pw143782 [PATCH 08/11] drivers: replace logging helpers In-Reply-To: <20240907145433.1479091-9-david.marchand@redhat.com> References: <20240907145433.1479091-9-david.marchand@redhat.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: David Marchand Message-Id: <20240907145924.E14EF121D10@dpdk.org> Date: Sat, 7 Sep 2024 16:59:24 +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/143782 _coding style issues_ ERROR:SPACING: space required after that ',' (ctx:VxB) #319: 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) #320: FILE: drivers/bus/platform/private.h:39: + RTE_FMT_TAIL(__VA_ARGS__,))) ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #598: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:340: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #642: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:366: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #647: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:371: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #655: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:376: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #675: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:391: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #694: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:405: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #713: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:419: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #726: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:424: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #731: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:429: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #759: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:439: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #764: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:444: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #772: FILE: drivers/compress/nitrox/nitrox_comp_reqmgr.c:452: + NITROX_LOG_LINE(DEBUG,); ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #1231: FILE: drivers/dma/odm/odm.h:71: + RTE_FMT("%s(): %u" RTE_FMT_HEAD(__VA_ARGS__,) " ", __func__, __LINE__, \ ^ CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3444: FILE: drivers/net/axgbe/axgbe_ethdev.c:1777: + "flags = 0x%x nsec = %"PRIu64, CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3460: 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'? #3651: 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'? #3660: 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'? #3696: FILE: drivers/net/axgbe/axgbe_i2c.c:324: + PMD_DRV_LOG_LINE(ERR, "failed to disable i2c master"); CHECK:CAMELCASE: Avoid CamelCase: #10467: FILE: drivers/net/ena/ena_ethdev.c:2887: + "mbuf[%" PRIu32 "] has unsupported offloads flags set: 0x%" PRIu64, CHECK:CAMELCASE: Avoid CamelCase: #10478: 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 #10996: FILE: drivers/net/ena/ena_rss.c:427: + "Setting RSS hash fields is not supported. Using default values: 0x%"PRIx64, total: 15 errors, 3 warnings, 5 checks, 10130 lines checked