[PATCH 1/3] net/sfc: split link update function

[Denis Pryazhennikov]

Separate the original link update function into
two functions: state retrieval and update.
This improves code clarity and maintainability.

Signed-off-by: Denis Pryazhennikov <denis.pryazhenni...@arknetworks.am>
Reviewed-by: Ivan Malov <ivan.ma...@arknetworks.am>
Reviewed-by: Andy Moreton <amore...@xilinx.com>
---
 drivers/net/sfc/sfc_ethdev.c | 28 +++++++++++++++++++---------
 1 file changed, 19 insertions(+), 9 deletions(-)

diff --git a/drivers/net/sfc/sfc_ethdev.c b/drivers/net/sfc/sfc_ethdev.c
index 320043145495..6d41eb000345 100644
--- a/drivers/net/sfc/sfc_ethdev.c
+++ b/drivers/net/sfc/sfc_ethdev.c
@@ -248,28 +248,38 @@ sfc_dev_start(struct rte_eth_dev *dev)
        return -rc;
 }
 
-static int
-sfc_dev_link_update(struct rte_eth_dev *dev, int wait_to_complete)
+static void
+sfc_dev_get_rte_link(struct rte_eth_dev *dev, int wait_to_complete,
+                    struct rte_eth_link *link)
 {
        struct sfc_adapter *sa = sfc_adapter_by_eth_dev(dev);
-       struct rte_eth_link current_link;
-       int ret;
 
-       sfc_log_init(sa, "entry");
+       SFC_ASSERT(link != NULL);
 
        if (sa->state != SFC_ETHDEV_STARTED) {
-               sfc_port_link_mode_to_info(EFX_LINK_UNKNOWN, &current_link);
+               sfc_port_link_mode_to_info(EFX_LINK_UNKNOWN, link);
        } else if (wait_to_complete) {
                efx_link_mode_t link_mode;
 
                if (efx_port_poll(sa->nic, &link_mode) != 0)
                        link_mode = EFX_LINK_UNKNOWN;
-               sfc_port_link_mode_to_info(link_mode, &current_link);
-
+               sfc_port_link_mode_to_info(link_mode, link);
        } else {
                sfc_ev_mgmt_qpoll(sa);
-               rte_eth_linkstatus_get(dev, &current_link);
+               rte_eth_linkstatus_get(dev, link);
        }
+}
+
+static int
+sfc_dev_link_update(struct rte_eth_dev *dev, int wait_to_complete)
+{
+       struct sfc_adapter *sa = sfc_adapter_by_eth_dev(dev);
+       struct rte_eth_link current_link;
+       int ret;
+
+       sfc_log_init(sa, "entry");
+
+       sfc_dev_get_rte_link(dev, wait_to_complete, &current_link);
 
        ret = rte_eth_linkstatus_set(dev, &current_link);
        if (ret == 0)
-- 
2.37.0 (Apple Git-136)