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 E967742528 for ; Wed, 6 Sep 2023 11:36:02 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id EFBDB4027E; Wed, 6 Sep 2023 11:36:02 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id AE8224027E for ; Wed, 6 Sep 2023 11:36:01 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id A578F1210A9; Wed, 6 Sep 2023 11:36:01 +0200 (CEST) Subject: |WARNING| pw131187 [PATCH v3 3/9] net/cpfl: add FXP low level implementation In-Reply-To: <20230906093407.3635038-4-wenjing.qiao@intel.com> References: <20230906093407.3635038-4-wenjing.qiao@intel.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Wenjing Qiao Message-Id: <20230906093601.A578F1210A9@dpdk.org> Date: Wed, 6 Sep 2023 11:36:01 +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/131187 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p' - possible side-effects? #198: FILE: drivers/net/cpfl/cpfl_actions.h:108: +#define CPFL_ACT_PREC_CHECK(p) ((p) > 0 && (p) <= CPFL_ACT_PREC_MAX) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'idx' - possible side-effects? #270: FILE: drivers/net/cpfl/cpfl_actions.h:180: +#define CPFL_ACT_MAKE_8B(prec, idx, val) \ + ((CPFL_ACT_PREC_SET(prec)) | CPFL_ACT_OP_8B | \ + (((idx) << CPFL_ACT_8B_A_INDEX_S) & CPFL_ACT_8B_A_INDEX_M) | \ + (((idx) << CPFL_ACT_8B_B_INDEX_S) & CPFL_ACT_8B_B_INDEX_M) | \ + (((val) << CPFL_ACT_8B_A_VAL_S) & CPFL_ACT_8B_A_VAL_M) | \ + (((val) << CPFL_ACT_8B_B_VAL_S) & CPFL_ACT_8B_B_VAL_M)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'val' - possible side-effects? #270: FILE: drivers/net/cpfl/cpfl_actions.h:180: +#define CPFL_ACT_MAKE_8B(prec, idx, val) \ + ((CPFL_ACT_PREC_SET(prec)) | CPFL_ACT_OP_8B | \ + (((idx) << CPFL_ACT_8B_A_INDEX_S) & CPFL_ACT_8B_A_INDEX_M) | \ + (((idx) << CPFL_ACT_8B_B_INDEX_S) & CPFL_ACT_8B_B_INDEX_M) | \ + (((val) << CPFL_ACT_8B_A_VAL_S) & CPFL_ACT_8B_A_VAL_M) | \ + (((val) << CPFL_ACT_8B_B_VAL_S) & CPFL_ACT_8B_B_VAL_M)) WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'? #1433: FILE: drivers/net/cpfl/cpfl_rules.c:37: + context |= SHIFT_VAL64(cmn_cfg->cache_wr_thru, WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'? #1434: FILE: drivers/net/cpfl/cpfl_rules.c:38: + MEV_RULE_CACHE_WR_THRU); WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'? #1613: FILE: drivers/net/cpfl/cpfl_rules.h:85: +#define MEV_RULE_CACHE_WR_THRU_S 21 WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'? #1614: FILE: drivers/net/cpfl/cpfl_rules.h:86: +#define MEV_RULE_CACHE_WR_THRU_M \ WARNING:TYPO_SPELLING: 'THRU' may be misspelled - perhaps 'THROUGH'? #1615: FILE: drivers/net/cpfl/cpfl_rules.h:87: + MAKE_MASK64(0x1, MEV_RULE_CACHE_WR_THRU_S) WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'? #1754: FILE: drivers/net/cpfl/cpfl_rules.h:226: + uint8_t cache_wr_thru; WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'? #1801: FILE: drivers/net/cpfl/cpfl_rules.h:273: + uint8_t cache_wr_thru, WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'? #1816: FILE: drivers/net/cpfl/cpfl_rules.h:288: + cfg_cmn->cache_wr_thru = cache_wr_thru; WARNING:TYPO_SPELLING: 'thru' may be misspelled - perhaps 'through'? #1816: FILE: drivers/net/cpfl/cpfl_rules.h:288: + cfg_cmn->cache_wr_thru = cache_wr_thru; total: 0 errors, 9 warnings, 3 checks, 1734 lines checked