[PATCH v2 net-next 2/5] ionic: use wait_on_bit_lock() rather than open code

[Shannon Nelson]

Replace the open-coded ionic_wait_for_bit() with the
kernel's wait_on_bit_lock().

Fixes: beead698b1736 ("ionic: Add the basic NDO callbacks for netdev support")
Signed-off-by: Shannon Nelson <snel...@pensando.io>
---
 drivers/net/ethernet/pensando/ionic/ionic_ethtool.c | 12 ++++++++----
 drivers/net/ethernet/pensando/ionic/ionic_lif.c     |  5 +++--
 drivers/net/ethernet/pensando/ionic/ionic_lif.h     |  9 ++-------
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/drivers/net/ethernet/pensando/ionic/ionic_ethtool.c 
b/drivers/net/ethernet/pensando/ionic/ionic_ethtool.c
index 7d10265f782a..7760fcd709b4 100644
--- a/drivers/net/ethernet/pensando/ionic/ionic_ethtool.c
+++ b/drivers/net/ethernet/pensando/ionic/ionic_ethtool.c
@@ -453,6 +453,7 @@ static int ionic_set_ringparam(struct net_device *netdev,
 {
        struct ionic_lif *lif = netdev_priv(netdev);
        bool running;
+       int err;
 
        if (ring->rx_mini_pending || ring->rx_jumbo_pending) {
                netdev_info(netdev, "Changing jumbo or mini descriptors not 
supported\n");
@@ -470,8 +471,9 @@ static int ionic_set_ringparam(struct net_device *netdev,
            ring->rx_pending == lif->nrxq_descs)
                return 0;
 
-       if (!ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET))
-               return -EBUSY;
+       err = ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET);
+       if (err)
+               return err;
 
        running = test_bit(IONIC_LIF_UP, lif->state);
        if (running)
@@ -504,6 +506,7 @@ static int ionic_set_channels(struct net_device *netdev,
 {
        struct ionic_lif *lif = netdev_priv(netdev);
        bool running;
+       int err;
 
        if (!ch->combined_count || ch->other_count ||
            ch->rx_count || ch->tx_count)
@@ -512,8 +515,9 @@ static int ionic_set_channels(struct net_device *netdev,
        if (ch->combined_count == lif->nxqs)
                return 0;
 
-       if (!ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET))
-               return -EBUSY;
+       err = ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET);
+       if (err)
+               return err;
 
        running = test_bit(IONIC_LIF_UP, lif->state);
        if (running)
diff --git a/drivers/net/ethernet/pensando/ionic/ionic_lif.c 
b/drivers/net/ethernet/pensando/ionic/ionic_lif.c
index 72107a0627a9..4d5883a7e586 100644
--- a/drivers/net/ethernet/pensando/ionic/ionic_lif.c
+++ b/drivers/net/ethernet/pensando/ionic/ionic_lif.c
@@ -1619,8 +1619,9 @@ int ionic_reset_queues(struct ionic_lif *lif)
        /* Put off the next watchdog timeout */
        netif_trans_update(lif->netdev);
 
-       if (!ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET))
-               return -EBUSY;
+       err = ionic_wait_for_bit(lif, IONIC_LIF_QUEUE_RESET);
+       if (err)
+               return err;
 
        running = netif_running(lif->netdev);
        if (running)
diff --git a/drivers/net/ethernet/pensando/ionic/ionic_lif.h 
b/drivers/net/ethernet/pensando/ionic/ionic_lif.h
index 812190e729c2..b74f7e9ee82d 100644
--- a/drivers/net/ethernet/pensando/ionic/ionic_lif.h
+++ b/drivers/net/ethernet/pensando/ionic/ionic_lif.h
@@ -185,15 +185,10 @@ struct ionic_lif {
 #define lif_to_txq(lif, i)     (&lif_to_txqcq((lif), i)->q)
 #define lif_to_rxq(lif, i)     (&lif_to_txqcq((lif), i)->q)
 
+/* return 0 if successfully set the bit, else non-zero */
 static inline int ionic_wait_for_bit(struct ionic_lif *lif, int bitname)
 {
-       unsigned long tlimit = jiffies + HZ;
-
-       while (test_and_set_bit(bitname, lif->state) &&
-              time_before(jiffies, tlimit))
-               usleep_range(100, 200);
-
-       return test_bit(bitname, lif->state);
+       return wait_on_bit_lock(lif->state, bitname, TASK_INTERRUPTIBLE);
 }
 
 static inline u32 ionic_coal_usec_to_hw(struct ionic *ionic, u32 usecs)
-- 
2.17.1