summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarsh Kumar <harsh1kumar@gmail.com>2012-09-26 23:36:15 +0530
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-09-26 14:02:54 -0700
commitee692cfad6520f7cc60b766a3c013f9e6eba6e38 (patch)
tree1cf190be878cf86cfd13cf6cea3d0a1a594cbacd
parent926ae525111021a3fa20f26b391d091d6c945275 (diff)
downloadop-kernel-dev-ee692cfad6520f7cc60b766a3c013f9e6eba6e38.zip
op-kernel-dev-ee692cfad6520f7cc60b766a3c013f9e6eba6e38.tar.gz
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>
-rw-r--r--drivers/staging/winbond/wb35tx.c7
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");
}
OpenPOWER on IntegriCloud