diff options
author | Jesper Juhl <jj@chaosbits.net> | 2012-04-09 22:51:58 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-04-18 16:37:05 -0700 |
commit | a6c8ef9526f149ce884b36841cc2e17ca890f1a4 (patch) | |
tree | c3439b4f59fd38f03ae6ada3fe341024a3c3d543 /drivers/staging | |
parent | 234bb3c60f1f1489630750aba4adf40154e0bd70 (diff) |
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>
Diffstat (limited to 'drivers/staging')
-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 763e028a5cc5..ee5261a36886 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)); |