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 7EEBDA0553 for ; Mon, 5 Sep 2022 13:00:14 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 52A3F4282B; Mon, 5 Sep 2022 13:00:14 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id A7C8940A7A for ; Mon, 5 Sep 2022 13:00:12 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 9FD94121F02; Mon, 5 Sep 2022 13:00:12 +0200 (CEST) Subject: |WARNING| pw115893 [PATCH v2 02/14] net/idpf/base: add logs and OS specific implementation In-Reply-To: <20220905105828.3190335-3-junfeng.guo@intel.com> References: <20220905105828.3190335-3-junfeng.guo@intel.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Junfeng Guo Message-Id: <20220905110012.9FD94121F02@dpdk.org> Date: Mon, 5 Sep 2022 13:00:12 +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/115893 _coding style issues_ WARNING:TYPO_SPELLING: 'definations' may be misspelled - perhaps 'definitions'? #70: Add some MACRO definations and small functions which are specific WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(arr) #187: FILE: drivers/net/idpf/base/iecm_osdep.h:100: +#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof(arr[0])) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'h' - possible side-effects? #195: FILE: drivers/net/idpf/base/iecm_osdep.h:108: +#define iecm_debug(h, m, s, ...) \ + do { \ + if (((m) & (h)->debug_mask)) \ + PMD_DRV_LOG_RAW(DEBUG, "iecm %02x.%x " s, \ + (h)->bus.device, (h)->bus.func, \ + ##__VA_ARGS__); \ + } while (0) CHECK:CAMELCASE: Avoid CamelCase: #213: FILE: drivers/net/idpf/base/iecm_osdep.h:126: + "0x%04X 0x%016"PRIx64" ", \ CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #213: FILE: drivers/net/idpf/base/iecm_osdep.h:126: + "0x%04X 0x%016"PRIx64" ", \ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'c' - possible side-effects? #279: FILE: drivers/net/idpf/base/iecm_osdep.h:192: +#define iecm_memdup(a, b, c, d) rte_memcpy(iecm_malloc(a, c), b, c) CHECK:CAMELCASE: Avoid CamelCase: #336: FILE: drivers/net/idpf/base/iecm_osdep.h:249: + snprintf(z_name, sizeof(z_name), "iecm_dma_%"PRIu64, rte_rand()); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #336: FILE: drivers/net/idpf/base/iecm_osdep.h:249: + snprintf(z_name, sizeof(z_name), "iecm_dma_%"PRIu64, rte_rand()); CHECK:MACRO_ARG_REUSE: Macro argument reuse 'd' - possible side-effects? #389: FILE: drivers/net/idpf/base/iecm_osdep.h:302: +#define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects? #402: FILE: drivers/net/idpf/base/iecm_osdep.h:315: +#define DIVIDE_AND_ROUND_UP(a, b) (((a) + (b) - 1) / (b)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects? #426: FILE: drivers/net/idpf/base/iecm_osdep.h:339: +#define IS_ZERO_ETHER_ADDR(addr) \ + (((bool)((((u16 *)(addr))[0] == ((u16)0x0)))) && \ + ((bool)((((u16 *)(addr))[1] == ((u16)0x0)))) && \ + ((bool)((((u16 *)(addr))[2] == ((u16)0x0))))) total: 0 errors, 2 warnings, 9 checks, 403 lines checked Warning in drivers/net/idpf/base/iecm_osdep.h: Using compiler attribute directly