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 CB41846419 for ; Wed, 19 Mar 2025 10:19:19 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 958EF402E8; Wed, 19 Mar 2025 10:19:19 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 450C04025A for ; Wed, 19 Mar 2025 10:19:18 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id 1692F1234FB; Wed, 19 Mar 2025 10:19:05 +0100 (CET) Subject: |WARNING| pw152475 [PATCH v5 14/14] net/zxdh: fix debugging errors In-Reply-To: <20250319085808.1912523-15-chen.bingbin@zte.com.cn> References: <20250319085808.1912523-15-chen.bingbin@zte.com.cn> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Bingbin Chen Message-Id: <20250319091905.1692F1234FB@dpdk.org> Date: Wed, 19 Mar 2025 10:19:05 +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/152475 _coding style issues_ CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'fld' may be better as '(fld)' to avoid precedence issues #137: FILE: drivers/net/zxdh/zxdh_common.h:68: +#define __zxdh_bit_sz(typ, fld) sizeof(__zxdh_nullp(typ)->fld) CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'fld' may be better as '(fld)' to avoid precedence issues #138: FILE: drivers/net/zxdh/zxdh_common.h:69: +#define __zxdh_bit_off(typ, fld) ((unsigned int)(uintptr_t) \ + (&(__zxdh_nullp(typ)->fld))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #140: FILE: drivers/net/zxdh/zxdh_common.h:71: +#define __zxdh_dw_bit_off(typ, fld) (32 - __zxdh_bit_sz(typ, fld) - \ + (__zxdh_bit_off(typ, fld) & 0x1f)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #140: FILE: drivers/net/zxdh/zxdh_common.h:71: +#define __zxdh_dw_bit_off(typ, fld) (32 - __zxdh_bit_sz(typ, fld) - \ + (__zxdh_bit_off(typ, fld) & 0x1f)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #144: FILE: drivers/net/zxdh/zxdh_common.h:75: +#define __zxdh_dw_mask(typ, fld) (__zxdh_mask(typ, fld) << \ + __zxdh_dw_bit_off(typ, fld)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #144: FILE: drivers/net/zxdh/zxdh_common.h:75: +#define __zxdh_dw_mask(typ, fld) (__zxdh_mask(typ, fld) << \ + __zxdh_dw_bit_off(typ, fld)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #148: FILE: drivers/net/zxdh/zxdh_common.h:79: +#define __zxdh_16_bit_off(typ, fld) (16 - __zxdh_bit_sz(typ, fld) - \ + (__zxdh_bit_off(typ, fld) & 0xf)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #148: FILE: drivers/net/zxdh/zxdh_common.h:79: +#define __zxdh_16_bit_off(typ, fld) (16 - __zxdh_bit_sz(typ, fld) - \ + (__zxdh_bit_off(typ, fld) & 0xf)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #151: FILE: drivers/net/zxdh/zxdh_common.h:82: +#define __zxdh_16_mask(typ, fld) (__zxdh_mask16(typ, fld) << \ + __zxdh_16_bit_off(typ, fld)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #151: FILE: drivers/net/zxdh/zxdh_common.h:82: +#define __zxdh_16_mask(typ, fld) (__zxdh_mask16(typ, fld) << \ + __zxdh_16_bit_off(typ, fld)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #163: FILE: drivers/net/zxdh/zxdh_common.h:94: +#define ZXDH_SET(typ, p, fld, v) do { \ + BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \ + uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \ + uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \ + zxdh_set_value(fld_sz, addr, v); \ +} while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #163: FILE: drivers/net/zxdh/zxdh_common.h:94: +#define ZXDH_SET(typ, p, fld, v) do { \ + BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \ + uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \ + uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \ + zxdh_set_value(fld_sz, addr, v); \ +} while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'typ' - possible side-effects? #170: FILE: drivers/net/zxdh/zxdh_common.h:101: +#define ZXDH_GET(typ, p, fld) ({ \ + BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \ + uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \ + uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \ + zxdh_get_value(fld_sz, addr); \ +}) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects? #170: FILE: drivers/net/zxdh/zxdh_common.h:101: +#define ZXDH_GET(typ, p, fld) ({ \ + BUILD_BUG_ON(__zxdh_bit_sz(typ, fld) % 8); \ + uint32_t fld_sz = __zxdh_bit_sz(typ, fld) / 8; \ + uint8_t *addr = ZXDH_ADDR_OF(typ, p, fld); \ + zxdh_get_value(fld_sz, addr); \ +}) total: 0 errors, 0 warnings, 14 checks, 2268 lines checked Warning in drivers/net/zxdh/zxdh_common.h: Writing to stdout or stderr Warning in drivers/net/zxdh/zxdh_common.h: Using __builtin helpers, prefer EAL macros