diff options
author | Harsh Kumar <harsh1kumar@gmail.com> | 2012-09-26 18:06:15 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-26 21:02:54 (GMT) |
commit | ee692cfad6520f7cc60b766a3c013f9e6eba6e38 (patch) | |
tree | 1cf190be878cf86cfd13cf6cea3d0a1a594cbacd /drivers/staging/winbond | |
parent | 926ae525111021a3fa20f26b391d091d6c945275 (diff) | |
download | linux-ee692cfad6520f7cc60b766a3c013f9e6eba6e38.tar.xz |
Staging: winbond: usb_free_urb(NULL) is safe
usb_free_urb(NULL) is safe. So, the check was removed.
Signed-off-by: Harsh Kumar <harsh1kumar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/winbond')
-rw-r--r-- | drivers/staging/winbond/wb35tx.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/staging/winbond/wb35tx.c b/drivers/staging/winbond/wb35tx.c index 474cad8..30a77cc 100644 --- a/drivers/staging/winbond/wb35tx.c +++ b/drivers/staging/winbond/wb35tx.c @@ -170,11 +170,8 @@ void Wb35Tx_destroy(struct hw_data *pHwData) } while ((pWb35Tx->EP2vm_state != VM_STOP) && (pWb35Tx->EP4vm_state != VM_STOP)); msleep(10); /* Delay for waiting function enter 940623.1.b */ - if (pWb35Tx->Tx4Urb) - usb_free_urb(pWb35Tx->Tx4Urb); - - if (pWb35Tx->Tx2Urb) - usb_free_urb(pWb35Tx->Tx2Urb); + usb_free_urb(pWb35Tx->Tx4Urb); + usb_free_urb(pWb35Tx->Tx2Urb); pr_debug("Wb35Tx_destroy OK\n"); } |