From: Sunil Kumar Kori <sk...@marvell.com>

Patch implements device close and reset operations for cn9k
and cn10k platforms.

Signed-off-by: Sunil Kumar Kori <sk...@marvell.com>
---
 drivers/net/cnxk/cnxk_ethdev.c | 35 ++++++++++++++++++++++++++++-------
 1 file changed, 28 insertions(+), 7 deletions(-)

diff --git a/drivers/net/cnxk/cnxk_ethdev.c b/drivers/net/cnxk/cnxk_ethdev.c
index 288e069..3929e7e 100644
--- a/drivers/net/cnxk/cnxk_ethdev.c
+++ b/drivers/net/cnxk/cnxk_ethdev.c
@@ -1154,6 +1154,9 @@ cnxk_nix_dev_start(struct rte_eth_dev *eth_dev)
        return rc;
 }
 
+static int cnxk_nix_dev_reset(struct rte_eth_dev *eth_dev);
+static int cnxk_nix_dev_close(struct rte_eth_dev *eth_dev);
+
 /* CNXK platform independent eth dev ops */
 struct eth_dev_ops cnxk_eth_dev_ops = {
        .mtu_set = cnxk_nix_mtu_set,
@@ -1165,6 +1168,8 @@ struct eth_dev_ops cnxk_eth_dev_ops = {
        .tx_queue_release = cnxk_nix_tx_queue_release,
        .rx_queue_release = cnxk_nix_rx_queue_release,
        .dev_stop = cnxk_nix_dev_stop,
+       .dev_close = cnxk_nix_dev_close,
+       .dev_reset = cnxk_nix_dev_reset,
        .tx_queue_start = cnxk_nix_tx_queue_start,
        .rx_queue_start = cnxk_nix_rx_queue_start,
        .rx_queue_stop = cnxk_nix_rx_queue_stop,
@@ -1304,7 +1309,7 @@ cnxk_eth_dev_init(struct rte_eth_dev *eth_dev)
 }
 
 static int
-cnxk_eth_dev_uninit(struct rte_eth_dev *eth_dev, bool mbox_close)
+cnxk_eth_dev_uninit(struct rte_eth_dev *eth_dev, bool reset)
 {
        struct cnxk_eth_dev *dev = cnxk_eth_pmd_priv(eth_dev);
        const struct eth_dev_ops *dev_ops = eth_dev->dev_ops;
@@ -1358,14 +1363,11 @@ cnxk_eth_dev_uninit(struct rte_eth_dev *eth_dev, bool 
mbox_close)
        rte_free(eth_dev->data->mac_addrs);
        eth_dev->data->mac_addrs = NULL;
 
-       /* Check if mbox close is needed */
-       if (!mbox_close)
-               return 0;
-
        rc = roc_nix_dev_fini(nix);
        /* Can be freed later by PMD if NPA LF is in use */
        if (rc == -EAGAIN) {
-               eth_dev->data->dev_private = NULL;
+               if (!reset)
+                       eth_dev->data->dev_private = NULL;
                return 0;
        } else if (rc) {
                plt_err("Failed in nix dev fini, rc=%d", rc);
@@ -1374,6 +1376,25 @@ cnxk_eth_dev_uninit(struct rte_eth_dev *eth_dev, bool 
mbox_close)
        return rc;
 }
 
+static int
+cnxk_nix_dev_close(struct rte_eth_dev *eth_dev)
+{
+       cnxk_eth_dev_uninit(eth_dev, false);
+       return 0;
+}
+
+static int
+cnxk_nix_dev_reset(struct rte_eth_dev *eth_dev)
+{
+       int rc;
+
+       rc = cnxk_eth_dev_uninit(eth_dev, true);
+       if (rc)
+               return rc;
+
+       return cnxk_eth_dev_init(eth_dev);
+}
+
 int
 cnxk_nix_remove(struct rte_pci_device *pci_dev)
 {
@@ -1384,7 +1405,7 @@ cnxk_nix_remove(struct rte_pci_device *pci_dev)
        eth_dev = rte_eth_dev_allocated(pci_dev->device.name);
        if (eth_dev) {
                /* Cleanup eth dev */
-               rc = cnxk_eth_dev_uninit(eth_dev, true);
+               rc = cnxk_eth_dev_uninit(eth_dev, false);
                if (rc)
                        return rc;
 
-- 
2.8.4

Reply via email to