staging: vt6656: code cleanup in device_netdev_ops' device_xmit()

Lindented, cleared checkpatch findings and simplified error handling.

Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

diff --git a/drivers/staging/vt6656/main_usb.c b/drivers/staging/vt6656/main_usb.c
index 217e8a62a603c38f2adfc168ac2d530fe982087a..0b95db43fc2ccb2ceafec6037ac3b4cf01c7bdfa 100644 (file)
--- a/drivers/staging/vt6656/main_usb.c
+++ b/drivers/staging/vt6656/main_usb.c
@@ -1306,39 +1306,37 @@ static int device_dma0_tx_80211(struct sk_buff *skb, struct net_device *dev)
        return NETDEV_TX_OK;
 }
 
-static int  device_xmit(struct sk_buff *skb, struct net_device *dev) {
-    PSDevice    pDevice=netdev_priv(dev);
-    struct net_device_stats* pStats = &pDevice->stats;
+static int device_xmit(struct sk_buff *skb, struct net_device *dev)
+{
+       PSDevice pDevice = netdev_priv(dev);
+       struct net_device_stats *stats = &pDevice->stats;
 
+       spin_lock_irq(&pDevice->lock);
 
-    spin_lock_irq(&pDevice->lock);
+       netif_stop_queue(dev);
 
-    netif_stop_queue(pDevice->dev);
+       if (!pDevice->bLinkPass) {
+               dev_kfree_skb_irq(skb);
+               goto out;
+       }
 
-    if (pDevice->bLinkPass == FALSE) {
-        dev_kfree_skb_irq(skb);
-        spin_unlock_irq(&pDevice->lock);
-        return 0;
-    }
-    if (pDevice->bStopDataPkt == TRUE) {
-        dev_kfree_skb_irq(skb);
-        pStats->tx_dropped++;
-        spin_unlock_irq(&pDevice->lock);
-        return 0;
-    }
+       if (pDevice->bStopDataPkt) {
+               dev_kfree_skb_irq(skb);
+               stats->tx_dropped++;
+               goto out;
+       }
 
-    if(nsDMA_tx_packet(pDevice, TYPE_AC0DMA, skb) !=0) {  //mike add:xmit fail!
-         if (netif_queue_stopped(pDevice->dev))
-              netif_wake_queue(pDevice->dev);
-    }
+       if (nsDMA_tx_packet(pDevice, TYPE_AC0DMA, skb)) {
+               if (netif_queue_stopped(dev))
+                       netif_wake_queue(dev);
+       }
 
-    spin_unlock_irq(&pDevice->lock);
+out:
+       spin_unlock_irq(&pDevice->lock);
 
-    return 0;
+       return NETDEV_TX_OK;
 }
 
-
-
 static unsigned const ethernet_polynomial = 0x04c11db7U;
 static inline u32 ether_crc(int length, unsigned char *data)
 {