From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <checkpatch@dpdk.org>
Received: by dpdk.org (Postfix, from userid 1017)
 id 2F8C52C50; Wed, 26 Apr 2017 12:36:42 +0200 (CEST)
In-Reply-To: <1493181063-14145-1-git-send-email-yulong.pei@intel.com>
References: <1493181063-14145-1-git-send-email-yulong.pei@intel.com>
To: test-report@dpdk.org
Cc: "Pei, Yulong" <yulong.pei@intel.com>
Message-Id: <20170426103642.2F8C52C50@dpdk.org>
Date: Wed, 26 Apr 2017 12:36:42 +0200 (CEST)
From: checkpatch@dpdk.org
Subject: [dpdk-test-report] |WARNING| pw23898 [PATCH] app/testpmd: initalize
	port_numa and ring_numa
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: <http://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: <http://dpdk.org/ml/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Wed, 26 Apr 2017 10:36:42 -0000

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

_coding style issues_


WARNING:TYPO_SPELLING: 'initalize' may be misspelled - perhaps 'initialize'?
#5: 
Subject: [dpdk-dev] [PATCH] app/testpmd: initalize port_numa and ring_numa

WARNING:TYPO_SPELLING: 'initalized' may be misspelled - perhaps 'initialized'?
#15: 
command line to enable numa, so port_numa and ring_numa were initalized

WARNING:TYPO_SPELLING: 'initalize' may be misspelled - perhaps 'initialize'?
#17: 
default, so port_numa and ring_numa also need to initalize by default,

ERROR:SPACING: space required after that ',' (ctx:VxV)
#53: FILE: app/test-pmd/testpmd.c:547:
+		memset(port_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#53: FILE: app/test-pmd/testpmd.c:547:
+		memset(port_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                               ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#54: FILE: app/test-pmd/testpmd.c:548:
+		memset(rxring_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                  ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#54: FILE: app/test-pmd/testpmd.c:548:
+		memset(rxring_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                                 ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#55: FILE: app/test-pmd/testpmd.c:549:
+		memset(txring_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                  ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#55: FILE: app/test-pmd/testpmd.c:549:
+		memset(txring_numa,NUMA_NO_CONFIG,RTE_MAX_ETHPORTS);
 		                                 ^

total: 6 errors, 3 warnings, 25 lines checked