diff options
author | Devendra Naga <devendra.aaru@gmail.com> | 2012-09-09 13:10:54 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-10 22:42:51 (GMT) |
commit | 89e6302ce7125d3be40637c2080be2b4728e4510 (patch) | |
tree | 9c9a308a40a22712494729b0b487f1968f946935 | |
parent | 3f5142583066d6d9a9484b060779dad419780720 (diff) | |
download | linux-fsl-qoriq-89e6302ce7125d3be40637c2080be2b4728e4510.tar.xz |
staging:wlan-ng: cleanup hfa384x_ctlxout_callback
goto done is not required and actually having goto done does
jumps the program to end of the function and calls return.
instead call return directly
Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/wlan-ng/hfa384x_usb.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/staging/wlan-ng/hfa384x_usb.c b/drivers/staging/wlan-ng/hfa384x_usb.c index 01a20d5..25dd263 100644 --- a/drivers/staging/wlan-ng/hfa384x_usb.c +++ b/drivers/staging/wlan-ng/hfa384x_usb.c @@ -3790,7 +3790,7 @@ static void hfa384x_ctlxout_callback(struct urb *urb) #endif if ((urb->status == -ESHUTDOWN) || (urb->status == -ENODEV) || (hw == NULL)) - goto done; + return; retry: spin_lock_irqsave(&hw->ctlxq.lock, flags); @@ -3803,7 +3803,7 @@ retry: */ if (list_empty(&hw->ctlxq.active)) { spin_unlock_irqrestore(&hw->ctlxq.lock, flags); - goto done; + return; } /* @@ -3886,9 +3886,6 @@ delresp: if (run_queue) hfa384x_usbctlxq_run(hw); - -done: - ; } /*---------------------------------------------------------------- |