From: Nikhil Vasoya <nikhil.vas...@chelsio.com>

Implement eth_dev_ops callback to get firmware version.

Signed-off-by: Nikhil Vasoya <nikhil.vas...@chelsio.com>
Signed-off-by: Rahul Lakkireddy <rahul.lakkire...@chelsio.com>
---
v2:
- Update FW version feature in cxgbe.ini and cxgbevf.ini

 doc/guides/nics/features/cxgbe.ini   |  1 +
 doc/guides/nics/features/cxgbevf.ini |  1 +
 drivers/net/cxgbe/cxgbe_ethdev.c     | 26 ++++++++++++++++++++++++++
 drivers/net/cxgbe/cxgbe_pfvf.h       |  2 ++
 drivers/net/cxgbe/cxgbevf_ethdev.c   |  1 +
 5 files changed, 31 insertions(+)

diff --git a/doc/guides/nics/features/cxgbe.ini 
b/doc/guides/nics/features/cxgbe.ini
index 6721740fbd..f41fc14825 100644
--- a/doc/guides/nics/features/cxgbe.ini
+++ b/doc/guides/nics/features/cxgbe.ini
@@ -26,6 +26,7 @@ Packet type parsing  = Y
 Basic stats          = Y
 Extended stats       = Y
 Stats per queue      = Y
+FW version           = Y
 EEPROM dump          = Y
 Registers dump       = Y
 Multiprocess aware   = Y
diff --git a/doc/guides/nics/features/cxgbevf.ini 
b/doc/guides/nics/features/cxgbevf.ini
index c8a25c9a8b..a3174ef399 100644
--- a/doc/guides/nics/features/cxgbevf.ini
+++ b/doc/guides/nics/features/cxgbevf.ini
@@ -22,6 +22,7 @@ Packet type parsing  = Y
 Basic stats          = Y
 Extended stats       = Y
 Stats per queue      = Y
+FW version           = Y
 Multiprocess aware   = Y
 Linux                = Y
 x86-32               = Y
diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c
index a6b5c0110a..304fafed25 100644
--- a/drivers/net/cxgbe/cxgbe_ethdev.c
+++ b/drivers/net/cxgbe/cxgbe_ethdev.c
@@ -1620,6 +1620,31 @@ static int cxgbe_fec_set(struct rte_eth_dev *dev, 
uint32_t fec_capa)
        return ret;
 }
 
+int cxgbe_fw_version_get(struct rte_eth_dev *dev, char *fw_version,
+                        size_t fw_size)
+{
+       struct port_info *pi = dev->data->dev_private;
+       struct adapter *adapter = pi->adapter;
+       int ret;
+
+       if (adapter->params.fw_vers == 0)
+               return -EIO;
+
+       ret = snprintf(fw_version, fw_size, "%u.%u.%u.%u",
+                      G_FW_HDR_FW_VER_MAJOR(adapter->params.fw_vers),
+                      G_FW_HDR_FW_VER_MINOR(adapter->params.fw_vers),
+                      G_FW_HDR_FW_VER_MICRO(adapter->params.fw_vers),
+                      G_FW_HDR_FW_VER_BUILD(adapter->params.fw_vers));
+       if (ret < 0)
+               return -EINVAL;
+
+       ret += 1;
+       if (fw_size < (size_t)ret)
+               return ret;
+
+       return 0;
+}
+
 static const struct eth_dev_ops cxgbe_eth_dev_ops = {
        .dev_start              = cxgbe_dev_start,
        .dev_stop               = cxgbe_dev_stop,
@@ -1665,6 +1690,7 @@ static const struct eth_dev_ops cxgbe_eth_dev_ops = {
        .fec_get_capability     = cxgbe_fec_get_capability,
        .fec_get                = cxgbe_fec_get,
        .fec_set                = cxgbe_fec_set,
+       .fw_version_get         = cxgbe_fw_version_get,
 };
 
 /*
diff --git a/drivers/net/cxgbe/cxgbe_pfvf.h b/drivers/net/cxgbe/cxgbe_pfvf.h
index 3c7aee0ae7..81d0fce2e5 100644
--- a/drivers/net/cxgbe/cxgbe_pfvf.h
+++ b/drivers/net/cxgbe/cxgbe_pfvf.h
@@ -63,4 +63,6 @@ int cxgbe_dev_xstats_get_names(struct rte_eth_dev *dev,
                               unsigned int n);
 int cxgbe_dev_xstats_get(struct rte_eth_dev *dev,
                         struct rte_eth_xstat *xstats, unsigned int n);
+int cxgbe_fw_version_get(struct rte_eth_dev *dev, char *fw_version,
+                        size_t fw_size);
 #endif /* _CXGBE_PFVF_H_ */
diff --git a/drivers/net/cxgbe/cxgbevf_ethdev.c 
b/drivers/net/cxgbe/cxgbevf_ethdev.c
index 4c809991b7..a62c56c2b9 100644
--- a/drivers/net/cxgbe/cxgbevf_ethdev.c
+++ b/drivers/net/cxgbe/cxgbevf_ethdev.c
@@ -92,6 +92,7 @@ static const struct eth_dev_ops cxgbevf_eth_dev_ops = {
        .xstats_get_names       = cxgbe_dev_xstats_get_names,
        .xstats_get_names_by_id = cxgbe_dev_xstats_get_names_by_id,
        .mac_addr_set           = cxgbe_mac_addr_set,
+       .fw_version_get         = cxgbe_fw_version_get,
 };
 
 /*
-- 
2.27.0

Reply via email to