From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id A294C1B463; Tue, 19 Jun 2018 23:31:52 +0200 (CEST) In-Reply-To: <20180619213058.12273-3-ajit.khaparde@broadcom.com> References: <20180619213058.12273-3-ajit.khaparde@broadcom.com> To: test-report@dpdk.org Cc: Ajit Khaparde Message-Id: <20180619213152.A294C1B463@dpdk.org> Date: Tue, 19 Jun 2018 23:31:52 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw41275 [02/31] net/bnxt: add Tx batching support 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: , X-List-Received-Date: Tue, 19 Jun 2018 21:31:52 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/41275 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'idx' - possible side-effects? #42: FILE: drivers/net/bnxt/bnxt_cpr.h:32: +#define NEXT_CMPL(cpr, idx, v, inc) do { \ + (idx) += (inc); \ + if (unlikely((idx) == (cpr)->cp_ring_struct->ring_size)) { \ + (v) = !(v); \ + idx = 0; \ + } \ +} while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'v' - possible side-effects? #42: FILE: drivers/net/bnxt/bnxt_cpr.h:32: +#define NEXT_CMPL(cpr, idx, v, inc) do { \ + (idx) += (inc); \ + if (unlikely((idx) == (cpr)->cp_ring_struct->ring_size)) { \ + (v) = !(v); \ + idx = 0; \ + } \ +} while (0) total: 0 errors, 0 warnings, 2 checks, 180 lines checked