From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <checkpatch@dpdk.org>
Received: by dpdk.org (Postfix, from userid 1017)
 id 070B31B718; Wed, 16 May 2018 04:09:57 +0200 (CEST)
In-Reply-To: <DB7PR08MB31637C042CB07826BE425E5F8F920@DB7PR08MB3163.eurprd08.prod.outlook.com>
References: <DB7PR08MB31637C042CB07826BE425E5F8F920@DB7PR08MB3163.eurprd08.prod.outlook.com>
To: test-report@dpdk.org
Cc: gavin hu <gavin.hu@arm.com>
Message-Id: <20180516020957.070B31B718@dpdk.org>
Date: Wed, 16 May 2018 04:09:57 +0200 (CEST)
From: checkpatch@dpdk.org
Subject: [dpdk-test-report] |WARNING| pw40074 [PATCH] dev Digest Vol 195
	Issue 45
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://dpdk.org/ml/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <https://dpdk.org/ml/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Wed, 16 May 2018 02:09:57 -0000

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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#31: 
When replying, please edit your Subject line so it is more specific than "Re: Contents of dev digest..."

ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 0123456789ab ("commit description")'
#95: 
commit 55fbc92d7800100628579643c9ee2770614fef10

WARNING:BAD_SIGN_OFF: Do not use whitespace before Reported-by:
#114: 
    Reported-by: Andrew Rybchenko <arybchenko@solarflare.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Signed-off-by:
#115: 
    Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Acked-by:
#116: 
    Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

WARNING:BAD_SIGN_OFF: Duplicate signature
#234: 
To: Gavin Hu <gavin.hu@arm.com>

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#334: FILE: drivers/net/tap/rte_eth_tap.c:506:
+if (txq->type == ETH_TUNTAP_TYPE_TUN) {
[...]
+char *buff_data = rte_pktmbuf_mtod(seg, void *);

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#378: FILE: drivers/net/tap/rte_eth_tap.c:1433:
+if (pmd->type == ETH_TUNTAP_TYPE_TAP) {
 if (is_zero_ether_addr(mac_addr))

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#387: FILE: drivers/net/tap/rte_eth_tap.c:1452:
+if (pmd->type == ETH_TUNTAP_TYPE_TAP) {
 memset(&ifr, 0, sizeof(struct ifreq));

total: 1 errors, 8 warnings, 0 checks, 130 lines checked