From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v3 2/4] net/e1000: add firmware version get Date: Tue, 3 Jan 2017 15:02:51 +0000 Message-ID: References: <1481008582-69416-1-git-send-email-qiming.yang@intel.com> <1482841816-54143-1-git-send-email-qiming.yang@intel.com> <1482841816-54143-3-git-send-email-qiming.yang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: remy.horton@intel.com To: Qiming Yang , dev@dpdk.org, thomas.monjalon@6wind.com Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id DF6BA5592 for ; Tue, 3 Jan 2017 16:02:53 +0100 (CET) In-Reply-To: <1482841816-54143-3-git-send-email-qiming.yang@intel.com> 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 12/27/2016 12:30 PM, Qiming Yang wrote: > This patch adds a new function eth_igb_fw_version_get. > > Signed-off-by: Qiming Yang > --- > v3 changes: > * use eth_igb_fw_version_get(struct rte_eth_dev *dev, u32 *fw_major, > u32 *fw_minor, u32 *fw_minor, u32 *fw_patch, u32 *etrack_id) instead > of eth_igb_fw_version_get(struct rte_eth_dev *dev, char *fw_version, > int fw_length). Add statusment in /doc/guides/nics/features/igb.ini. > --- > --- > doc/guides/nics/features/igb.ini | 1 + > drivers/net/e1000/igb_ethdev.c | 43 ++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 44 insertions(+) > > diff --git a/doc/guides/nics/features/igb.ini b/doc/guides/nics/features/igb.ini > index 9fafe72..ffd87ba 100644 > --- a/doc/guides/nics/features/igb.ini > +++ b/doc/guides/nics/features/igb.ini > @@ -39,6 +39,7 @@ EEPROM dump = Y > Registers dump = Y > BSD nic_uio = Y > Linux UIO = Y > +FW version = Y Please keep same location with default.ini file. Why you are putting this just into middle of the uio and vfio? > Linux VFIO = Y > x86-32 = Y > x86-64 = Y > diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/igb_ethdev.c > index 4a15447..25344b7 100644 > --- a/drivers/net/e1000/igb_ethdev.c > +++ b/drivers/net/e1000/igb_ethdev.c > @@ -120,6 +120,8 @@ static int eth_igb_xstats_get_names(struct rte_eth_dev *dev, > unsigned limit); > static void eth_igb_stats_reset(struct rte_eth_dev *dev); > static void eth_igb_xstats_reset(struct rte_eth_dev *dev); > +static void eth_igb_fw_version_get(struct rte_eth_dev *dev, u32 *fw_major, > + u32 *fw_minor, u32 *fw_patch, u32 *etrack_id); I think you can use a struct as parameter here. But beware, that struct should NOT be a public struct. > static void eth_igb_infos_get(struct rte_eth_dev *dev, > struct rte_eth_dev_info *dev_info); > static const uint32_t *eth_igb_supported_ptypes_get(struct rte_eth_dev *dev); > @@ -389,6 +391,7 @@ static const struct eth_dev_ops eth_igb_ops = { > .xstats_get_names = eth_igb_xstats_get_names, > .stats_reset = eth_igb_stats_reset, > .xstats_reset = eth_igb_xstats_reset, > + .fw_version_get = eth_igb_fw_version_get, > .dev_infos_get = eth_igb_infos_get, > .dev_supported_ptypes_get = eth_igb_supported_ptypes_get, > .mtu_set = eth_igb_mtu_set, > @@ -1981,6 +1984,46 @@ eth_igbvf_stats_reset(struct rte_eth_dev *dev) > } > <...>