[media] staging: as102: Remove redundant NULL check before release_firmware() and...
authorJesper Juhl <jj@chaosbits.net>
Mon, 9 Apr 2012 19:51:48 +0000 (16:51 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 19 Apr 2012 12:09:13 +0000 (09:09 -0300)
release_firmware() deals gracefullt with NULL pointers - it's
redundant to check for them before calling the function.

Also remove a few pointless comments - it's rather obvious from the
code that kfree() free's a buffer and that release_firmware() releases
firmware - comments just stating that add no value.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/staging/media/as102/as102_fw.c

index 1075fb1df0d9c04a967634e56eb2e5a581d3421b..b9670ee41b4ec3b59c07f72a514ab57a29ac4942 100644 (file)
@@ -230,11 +230,8 @@ int as102_fw_upload(struct as10x_bus_adapter_t *bus_adap)
        pr_info("%s: firmware: %s loaded with success\n",
                DRIVER_NAME, fw2);
 error:
-       /* free data buffer */
        kfree(cmd_buf);
-       /* release firmware if needed */
-       if (firmware != NULL)
-               release_firmware(firmware);
+       release_firmware(firmware);
 
        LEAVE();
        return errno;