From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v3] app/testpmd: add new command for show port info Date: Wed, 29 Aug 2018 17:24:25 +0100 Message-ID: References: <1535536120-6934-1-git-send-email-emma.finn@intel.com> <1535539649-7385-1-git-send-email-emma.finn@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: john.mcnamara@intel.com, bernard.iremonger@intel.com, stephen@networkplumber.org To: Emma Finn , dev@dpdk.org Return-path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id A74D65A for ; Wed, 29 Aug 2018 18:24:28 +0200 (CEST) In-Reply-To: <1535539649-7385-1-git-send-email-emma.finn@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 8/29/2018 11:47 AM, Emma Finn wrote: > existing testpmd command "show port info" is too verbose. > Added a new summary command to print brief information on ports. > > console output: > testpmd> show port summary all > Number of available ports: 2 > Port MAC Address Name Driver Status Link > 0 11:22:33:44:55:66 0000:07:00.0, net_i40e, up, 40000 Mbps > 1 66:55:44:33:22:11 0000:07:00.1, net_i40e, up, 40000 Mbps > > Signed-off-by: Emma Finn When header is used, I think no need "," between fields. Except from this, Reviewed-by: Ferruh Yigit