staging: bcm: Cleanup real_int_callback()
authorJade Bilkey <herself@thefumon.com>
Mon, 17 Mar 2014 23:45:25 +0000 (19:45 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 18 Mar 2014 18:48:42 +0000 (11:48 -0700)
Fixes checkpatch.pl warnings and errors and removes some dead code.

Checkpath warnings and errors fixed:

WARNING: line over 80 characters
WARNING: space prohibited before semicolon

ERROR: that open brace { should be on the previous line
ERROR: space required before the open parenthesis '('
ERROR: space required after that ',' (ctx:VxV)

Signed-off-by: Jade Bilkey <herself@thefumon.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/InterfaceIsr.c

index 7b39f4f..1ad80c6 100644 (file)
@@ -4,28 +4,30 @@
 static void read_int_callback(struct urb *urb/*, struct pt_regs *regs*/)
 {
        int             status = urb->status;
-       struct bcm_interface_adapter *psIntfAdapter = (struct bcm_interface_adapter *)urb->context;
-       struct bcm_mini_adapter *Adapter = psIntfAdapter->psAdapter ;
+       struct bcm_interface_adapter *psIntfAdapter =
+               (struct bcm_interface_adapter *)urb->context;
+       struct bcm_mini_adapter *Adapter = psIntfAdapter->psAdapter;
 
        if (netif_msg_intr(Adapter))
                pr_info(PFX "%s: interrupt status %d\n",
-                       Adapter->dev->name, status);
+                               Adapter->dev->name, status);
 
-       if(Adapter->device_removed == TRUE)
-       {
-               BCM_DEBUG_PRINT(Adapter,DBG_TYPE_OTHERS, INTF_INIT, DBG_LVL_ALL,"Device has Got Removed.");
-               return ;
+       if (Adapter->device_removed == TRUE) {
+               BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, INTF_INIT,
+                               DBG_LVL_ALL, "Device has Got Removed.");
+               return;
        }
 
-       if(((Adapter->bPreparingForLowPowerMode == TRUE) && (Adapter->bDoSuspend == TRUE)) ||
-               psIntfAdapter->bSuspended ||
-               psIntfAdapter->bPreparingForBusSuspend)
-       {
-                       BCM_DEBUG_PRINT(Adapter,DBG_TYPE_OTHERS, INTF_INIT, DBG_LVL_ALL,"Interrupt call back is called while suspending the device");
-                       return ;
+       if (((Adapter->bPreparingForLowPowerMode == TRUE) &&
+                               (Adapter->bDoSuspend == TRUE)) ||
+                       psIntfAdapter->bSuspended ||
+                       psIntfAdapter->bPreparingForBusSuspend) {
+               BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, INTF_INIT,
+                               DBG_LVL_ALL,
+                               "Interrupt call back is called while suspending the device");
+               return;
        }
 
-       //BCM_DEBUG_PRINT(Adapter,DBG_TYPE_TX, NEXT_SEND, DBG_LVL_ALL, "interrupt urb status %d", status);
        switch (status) {
            /* success */
            case STATUS_SUCCESS: