From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id B7F6BA0577 for ; Tue, 14 Apr 2020 10:16:18 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 089B81C1C3; Tue, 14 Apr 2020 10:16:11 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 5D05E1C1BE; Tue, 14 Apr 2020 10:16:10 +0200 (CEST) In-Reply-To: <1586852011-37536-5-git-send-email-venkatkumar.duvvuru@broadcom.com> References: <1586852011-37536-5-git-send-email-venkatkumar.duvvuru@broadcom.com> To: test-report@dpdk.org Cc: Venkat Duvvuru Message-Id: <20200414081610.5D05E1C1BE@dpdk.org> Date: Tue, 14 Apr 2020 10:16:10 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw68390 [PATCH v3 04/34] net/bnxt: add initial tf core session open 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/68390 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'parms' - possible side-effects? #1794: FILE: drivers/net/bnxt/tf_core/tf_msg_common.h:14: +#define MSG_PREP(parms, mb, type, subtype, req, resp) do { \ + parms.mailbox = mb; \ + parms.tf_type = type; \ + parms.tf_subtype = subtype; \ + parms.tf_resp_code = 0; \ + parms.req_size = sizeof(req); \ + parms.req_data = (uint32_t *)&(req); \ + parms.resp_size = sizeof(resp); \ + parms.resp_data = (uint32_t *)&(resp); \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'parms' - possible side-effects? #1805: FILE: drivers/net/bnxt/tf_core/tf_msg_common.h:25: +#define MSG_PREP_NO_REQ(parms, mb, type, subtype, resp) do { \ + parms.mailbox = mb; \ + parms.tf_type = type; \ + parms.tf_subtype = subtype; \ + parms.tf_resp_code = 0; \ + parms.req_size = 0; \ + parms.req_data = NULL; \ + parms.resp_size = sizeof(resp); \ + parms.resp_data = (uint32_t *)&(resp); \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'parms' - possible side-effects? #1816: FILE: drivers/net/bnxt/tf_core/tf_msg_common.h:36: +#define MSG_PREP_NO_RESP(parms, mb, type, subtype, req) do { \ + parms.mailbox = mb; \ + parms.tf_type = type; \ + parms.tf_subtype = subtype; \ + parms.tf_resp_code = 0; \ + parms.req_size = sizeof(req); \ + parms.req_data = (uint32_t *)&(req); \ + parms.resp_size = 0; \ + parms.resp_data = NULL; \ + } while (0) total: 0 errors, 0 warnings, 3 checks, 2202 lines checked