Skip to content

Commit

Permalink
staging: vt6656: Don't needlessly test for NULL before release_firmwa…
Browse files Browse the repository at this point in the history
…re()

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>
  • Loading branch information
Jesper Juhl authored and Greg Kroah-Hartman committed Apr 18, 2012
1 parent 234bb3c commit a6c8ef9
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions drivers/staging/vt6656/main_usb.c
Original file line number Diff line number Diff line change
Expand Up @@ -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));
Expand Down

0 comments on commit a6c8ef9

Please sign in to comment.