From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id EF6F9A0096 for ; Thu, 6 Jun 2019 13:03:02 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E79F01B946; Thu, 6 Jun 2019 13:03:02 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id E96941B970; Thu, 6 Jun 2019 13:03:00 +0200 (CEST) In-Reply-To: <90c859a045f76c85e2d0911118a3b2d7718cd1a0.1559818024.git.xuanziyang2@huawei.com> References: <90c859a045f76c85e2d0911118a3b2d7718cd1a0.1559818024.git.xuanziyang2@huawei.com> To: test-report@dpdk.org Cc: Ziyang Xuan Message-Id: <20190606110300.E96941B970@dpdk.org> Date: Thu, 6 Jun 2019 13:03:00 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw54474 [PATCH v4 01/11] net/hinic/base: add registers for Huawei Hi1822 NIC X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/54474 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects? #117: FILE: drivers/net/hinic/base/hinic_csr.h:83: +#define HINIC_EQ_HI_PHYS_ADDR_REG(type, q_id, pg_num) \ + ((u32)((type == HINIC_AEQ) ? \ + HINIC_AEQ_HI_PHYS_ADDR_REG(q_id, pg_num) : \ + HINIC_CEQ_HI_PHYS_ADDR_REG(q_id, pg_num))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pg_num' - possible side-effects? #117: FILE: drivers/net/hinic/base/hinic_csr.h:83: +#define HINIC_EQ_HI_PHYS_ADDR_REG(type, q_id, pg_num) \ + ((u32)((type == HINIC_AEQ) ? \ + HINIC_AEQ_HI_PHYS_ADDR_REG(q_id, pg_num) : \ + HINIC_CEQ_HI_PHYS_ADDR_REG(q_id, pg_num))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q_id' - possible side-effects? #122: FILE: drivers/net/hinic/base/hinic_csr.h:88: +#define HINIC_EQ_LO_PHYS_ADDR_REG(type, q_id, pg_num) \ + ((u32)((type == HINIC_AEQ) ? \ + HINIC_AEQ_LO_PHYS_ADDR_REG(q_id, pg_num) : \ + HINIC_CEQ_LO_PHYS_ADDR_REG(q_id, pg_num))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pg_num' - possible side-effects? #122: FILE: drivers/net/hinic/base/hinic_csr.h:88: +#define HINIC_EQ_LO_PHYS_ADDR_REG(type, q_id, pg_num) \ + ((u32)((type == HINIC_AEQ) ? \ + HINIC_AEQ_LO_PHYS_ADDR_REG(q_id, pg_num) : \ + HINIC_CEQ_LO_PHYS_ADDR_REG(q_id, pg_num))) total: 0 errors, 0 warnings, 4 checks, 164 lines checked