From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <checkpatch@dpdk.org>
Received: by dpdk.org (Postfix, from userid 1017)
 id 5AE341BB0B; Fri, 29 Jun 2018 20:14:42 +0200 (CEST)
In-Reply-To: <ed3b702573e5f562600f6a446e1bbcdadd90246c.1530295732.git.rahul.lakkireddy@chelsio.com>
References: <ed3b702573e5f562600f6a446e1bbcdadd90246c.1530295732.git.rahul.lakkireddy@chelsio.com>
To: test-report@dpdk.org
Cc: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Message-Id: <20180629181442.5AE341BB0B@dpdk.org>
Date: Fri, 29 Jun 2018 20:14:42 +0200 (CEST)
From: checkpatch@dpdk.org
Subject: [dpdk-test-report] |WARNING| pw41988 [1/9] net/cxgbe: query
	firmware for HASH filter resources
X-BeenThere: test-report@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: automatic DPDK test reports <test-report.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Fri, 29 Jun 2018 18:14:42 -0000

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/41988

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#87: FILE: drivers/net/cxgbe/cxgbe_compat.h:257:
+#define mult_frac(x, numer, denom)(                     \
+{                                                       \
+	typeof(x) quot = (x) / (denom);                 \
+	typeof(x) rem  = (x) % (denom);                 \
+	(quot * (numer)) + ((rem * (numer)) / (denom)); \
+}                                                       \
+)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'numer' - possible side-effects?
#87: FILE: drivers/net/cxgbe/cxgbe_compat.h:257:
+#define mult_frac(x, numer, denom)(                     \
+{                                                       \
+	typeof(x) quot = (x) / (denom);                 \
+	typeof(x) rem  = (x) % (denom);                 \
+	(quot * (numer)) + ((rem * (numer)) / (denom)); \
+}                                                       \
+)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'denom' - possible side-effects?
#87: FILE: drivers/net/cxgbe/cxgbe_compat.h:257:
+#define mult_frac(x, numer, denom)(                     \
+{                                                       \
+	typeof(x) quot = (x) / (denom);                 \
+	typeof(x) rem  = (x) % (denom);                 \
+	(quot * (numer)) + ((rem * (numer)) / (denom)); \
+}                                                       \
+)

total: 0 errors, 0 warnings, 3 checks, 231 lines checked