Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2017-01-04 Thread Zhang, Helin
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Thursday, December 22, 2016 11:31 PM > To: Yigit, Ferruh > Cc: Yang, Qiming; dev@dpdk.org; Horton, Remy > Subject: Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: a

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-23 Thread Ferruh Yigit
On 12/22/2016 3:31 PM, Thomas Monjalon wrote: > 2016-12-22 15:05, Ferruh Yigit: >> On 12/22/2016 2:47 PM, Thomas Monjalon wrote: >>> 2016-12-22 14:36, Ferruh Yigit: On 12/22/2016 11:07 AM, Thomas Monjalon wrote: > I think it is OK to add a new dev_ops and a new API function for firmware >>

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-22 Thread Thomas Monjalon
2016-12-22 15:05, Ferruh Yigit: > On 12/22/2016 2:47 PM, Thomas Monjalon wrote: > > 2016-12-22 14:36, Ferruh Yigit: > >> On 12/22/2016 11:07 AM, Thomas Monjalon wrote: > >>> I think it is OK to add a new dev_ops and a new API function for firmware > >>> query. Generally speaking, it is a good thing

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-22 Thread Ferruh Yigit
On 12/22/2016 2:47 PM, Thomas Monjalon wrote: > 2016-12-22 14:36, Ferruh Yigit: >> On 12/22/2016 11:07 AM, Thomas Monjalon wrote: >>> I think it is OK to add a new dev_ops and a new API function for firmware >>> query. Generally speaking, it is a good thing to avoid putting all >>> informations in

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-22 Thread Thomas Monjalon
2016-12-22 14:36, Ferruh Yigit: > On 12/22/2016 11:07 AM, Thomas Monjalon wrote: > > I think it is OK to add a new dev_ops and a new API function for firmware > > query. Generally speaking, it is a good thing to avoid putting all > > informations in the same structure (e.g. rte_eth_dev_info). > >

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-22 Thread Ferruh Yigit
On 12/22/2016 11:07 AM, Thomas Monjalon wrote: > 2016-12-08 16:34, Remy Horton: >> >> On 06/12/2016 15:16, Qiming Yang wrote: >> [..] >>> Qiming Yang (5): >>> ethdev: add firmware version get >>> net/e1000: add firmware version get >>> net/ixgbe: add firmware version get >>> net/i40e: add f

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-22 Thread Thomas Monjalon
2016-12-08 16:34, Remy Horton: > > On 06/12/2016 15:16, Qiming Yang wrote: > [..] > > Qiming Yang (5): > > ethdev: add firmware version get > > net/e1000: add firmware version get > > net/ixgbe: add firmware version get > > net/i40e: add firmware version get > > ethtool: dispaly bus info

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-11 Thread Yang, Qiming
o: Yang, Qiming ; dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get On 06/12/2016 15:16, Qiming Yang wrote: [..] > Qiming Yang (5): > ethdev: add firmware version get > net/e1000: add firmware version get > net/ixgbe: add firmware ve

Re: [dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-08 Thread Remy Horton
On 06/12/2016 15:16, Qiming Yang wrote: [..] Qiming Yang (5): ethdev: add firmware version get net/e1000: add firmware version get net/ixgbe: add firmware version get net/i40e: add firmware version get ethtool: dispaly bus info and firmware version s/dispaly/display doc/guides/rel_n

[dpdk-dev] [PATCH v2 0/5] example/ethtool: add bus info and fw version get

2016-12-05 Thread Qiming Yang
Now, the example ethtool can only show the driver information. From customers' point of view, it should be better if we can have the same way that the Linux kernel ethtool does to show the bus-info and firmware-version. These five patches add a new API to fetch firmware version and implement the d