[PATCH] r8152: Use guard clause and fix comment typos

[Prashant Malani]

Use a guard clause in tx_bottom() to reduce the indentation of the
do-while loop. In doing so, convert the do-while to a while to make the
guard clause checks consistent.

Also, fix a couple of spelling and grammatical mistakes in the
r8152_csum_workaround() function comment.

Change-Id: I460befde150ad92248fd85b0f189ec2df2ab8431
Signed-off-by: Prashant Malani <pmal...@chromium.org>
Reviewed-by: Grant Grundler <grund...@chromium.org>
---
 drivers/net/usb/r8152.c | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
index 08726090570e1..82bc69a5b24fb 100644
--- a/drivers/net/usb/r8152.c
+++ b/drivers/net/usb/r8152.c
@@ -1688,7 +1688,7 @@ static struct tx_agg *r8152_get_tx_agg(struct r8152 *tp)
 }
 
 /* r8152_csum_workaround()
- * The hw limites the value the transport offset. When the offset is out of the
+ * The hw limits the value of the transport offset. When the offset is out of
  * range, calculate the checksum by sw.
  */
 static void r8152_csum_workaround(struct r8152 *tp, struct sk_buff *skb,
@@ -2177,8 +2177,9 @@ static void tx_bottom(struct r8152 *tp)
 {
        int res;
 
-       do {
+       while (1) {
                struct tx_agg *agg;
+               struct net_device *netdev = tp->netdev;
 
                if (skb_queue_empty(&tp->tx_queue))
                        break;
@@ -2188,26 +2189,25 @@ static void tx_bottom(struct r8152 *tp)
                        break;
 
                res = r8152_tx_agg_fill(tp, agg);
-               if (res) {
-                       struct net_device *netdev = tp->netdev;
+               if (!res)
+                       break;
 
-                       if (res == -ENODEV) {
-                               rtl_set_unplug(tp);
-                               netif_device_detach(netdev);
-                       } else {
-                               struct net_device_stats *stats = &netdev->stats;
-                               unsigned long flags;
+               if (res == -ENODEV) {
+                       rtl_set_unplug(tp);
+                       netif_device_detach(netdev);
+               } else {
+                       struct net_device_stats *stats = &netdev->stats;
+                       unsigned long flags;
 
-                               netif_warn(tp, tx_err, netdev,
-                                          "failed tx_urb %d\n", res);
-                               stats->tx_dropped += agg->skb_num;
+                       netif_warn(tp, tx_err, netdev,
+                                  "failed tx_urb %d\n", res);
+                       stats->tx_dropped += agg->skb_num;
 
-                               spin_lock_irqsave(&tp->tx_lock, flags);
-                               list_add_tail(&agg->list, &tp->tx_free);
-                               spin_unlock_irqrestore(&tp->tx_lock, flags);
-                       }
+                       spin_lock_irqsave(&tp->tx_lock, flags);
+                       list_add_tail(&agg->list, &tp->tx_free);
+                       spin_unlock_irqrestore(&tp->tx_lock, flags);
                }
-       } while (res == 0);
+       }
 }
 
 static void bottom_half(unsigned long data)
-- 
2.23.0.351.gc4317032e6-goog