diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2012-04-09 20:51:58 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-04-18 23:37:05 (GMT) |
| commit | a6c8ef9526f149ce884b36841cc2e17ca890f1a4 (patch) | |
| tree | c3439b4f59fd38f03ae6ada3fe341024a3c3d543 | |
| parent | 234bb3c60f1f1489630750aba4adf40154e0bd70 (diff) | |
| download | linux-a6c8ef9526f149ce884b36841cc2e17ca890f1a4.tar.xz | |
staging: vt6656: Don't needlessly test for NULL before release_firmware()
Checking for a NULL pointer before calling release_firmware() is
redundant since the function does that check itself.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/vt6656/main_usb.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/vt6656/main_usb.c b/drivers/staging/vt6656/main_usb.c index 763e028..ee5261a 100644 --- a/drivers/staging/vt6656/main_usb.c +++ b/drivers/staging/vt6656/main_usb.c @@ -1257,9 +1257,7 @@ static void __devexit vt6656_disconnect(struct usb_interface *intf) } device_release_WPADEV(device); - - if (device->firmware) - release_firmware(device->firmware); + release_firmware(device->firmware); usb_set_intfdata(intf, NULL); usb_put_dev(interface_to_usbdev(intf)); |