[dpdk-dev,v4,2/5] net/e1000: add firmware version get
Checks
Commit Message
This patch adds a new function eth_igb_fw_version_get.
Signed-off-by: Qiming Yang <qiming.yang@intel.com>
---
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(+)
Comments
On 1/4/2017 12:03 PM, Qiming Yang wrote:
> This patch adds a new function eth_igb_fw_version_get.
>
> Signed-off-by: Qiming Yang <qiming.yang@intel.com>
> ---
<...>
>
> static void
> +eth_igb_fw_version_get(struct rte_eth_dev *dev, u32 *fw_major, u32 *fw_minor,
> + u32 *fw_patch, u32 *etrack_id)
> +{
<...>
> + default:
> + /* if option rom is valid, display its version too*/
> + if (fw.or_valid) {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + *etrack_id = fw.etrack_id;
> + *fw_patch = fw.or_patch;
> + /* no option rom */
> + } else {
> + if (fw.etrack_id != 0X0000) {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + *etrack_id = fw.etrack_id;
indentation is wrong here. Also it looks like major, minor assignment is
common and can be moved from if statement.
> + } else {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + }
> + }
> + break;
> + }
> +}
<...>
-----Original Message-----
From: Yigit, Ferruh
Sent: Wednesday, January 4, 2017 10:00 PM
To: Yang, Qiming <qiming.yang@intel.com>; dev@dpdk.org
Cc: Zhang, Helin <helin.zhang@intel.com>; Horton, Remy <remy.horton@intel.com>
Subject: Re: [PATCH v4 2/5] net/e1000: add firmware version get
On 1/4/2017 12:03 PM, Qiming Yang wrote:
> This patch adds a new function eth_igb_fw_version_get.
>
> Signed-off-by: Qiming Yang <qiming.yang@intel.com>
> ---
<...>
>
> static void
> +eth_igb_fw_version_get(struct rte_eth_dev *dev, u32 *fw_major, u32 *fw_minor,
> + u32 *fw_patch, u32 *etrack_id)
> +{
<...>
> + default:
> + /* if option rom is valid, display its version too*/
> + if (fw.or_valid) {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + *etrack_id = fw.etrack_id;
> + *fw_patch = fw.or_patch;
> + /* no option rom */
> + } else {
> + if (fw.etrack_id != 0X0000) {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + *etrack_id = fw.etrack_id;
indentation is wrong here. Also it looks like major, minor assignment is common and can be moved from if statement.
Qiming: will correct them
> + } else {
> + *fw_major = fw.eep_major;
> + *fw_minor = fw.eep_minor;
> + }
> + }
> + break;
> + }
> +}
<...>
@@ -35,6 +35,7 @@ Packet type parsing = Y
Timesync = Y
Basic stats = Y
Extended stats = Y
+FW version = Y
EEPROM dump = Y
Registers dump = Y
BSD nic_uio = Y
@@ -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);
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)
}
static void
+eth_igb_fw_version_get(struct rte_eth_dev *dev, u32 *fw_major, u32 *fw_minor,
+ u32 *fw_patch, u32 *etrack_id)
+{
+ struct e1000_hw *hw = E1000_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+ struct e1000_fw_version fw;
+
+ e1000_get_fw_version(hw, &fw);
+
+ switch (hw->mac.type) {
+ case e1000_i210:
+ case e1000_i211:
+ if (!(e1000_get_flash_presence_i210(hw))) {
+ *fw_major = fw.invm_major;
+ *fw_minor = fw.invm_minor;
+ break;
+ }
+ /* fall through */
+ default:
+ /* if option rom is valid, display its version too*/
+ if (fw.or_valid) {
+ *fw_major = fw.eep_major;
+ *fw_minor = fw.eep_minor;
+ *etrack_id = fw.etrack_id;
+ *fw_patch = fw.or_patch;
+ /* no option rom */
+ } else {
+ if (fw.etrack_id != 0X0000) {
+ *fw_major = fw.eep_major;
+ *fw_minor = fw.eep_minor;
+ *etrack_id = fw.etrack_id;
+ } else {
+ *fw_major = fw.eep_major;
+ *fw_minor = fw.eep_minor;
+ }
+ }
+ break;
+ }
+}
+
+static void
eth_igb_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
{
struct e1000_hw *hw = E1000_DEV_PRIVATE_TO_HW(dev->data->dev_private);