diff options
author | Jesper Juhl | 2012-04-09 21:51:48 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab | 2012-04-19 14:09:13 +0200 |
commit | b803cc58c33c0c5b6220150336a6f1c7958f1404 (patch) | |
tree | 9b169079f0e4a2a8bfbc641f44f48ac6e085edd5 /drivers/staging | |
parent | [media] v4l2-ctrls: fix integer overflow in try_set_ext_ctrls() (diff) | |
download | kernel-qcow2-linux-b803cc58c33c0c5b6220150336a6f1c7958f1404.tar.gz kernel-qcow2-linux-b803cc58c33c0c5b6220150336a6f1c7958f1404.tar.xz kernel-qcow2-linux-b803cc58c33c0c5b6220150336a6f1c7958f1404.zip |
[media] staging: as102: Remove redundant NULL check before release_firmware() and pointless comments
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>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/media/as102/as102_fw.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/media/as102/as102_fw.c b/drivers/staging/media/as102/as102_fw.c index 1075fb1df0d9..b9670ee41b4e 100644 --- a/drivers/staging/media/as102/as102_fw.c +++ b/drivers/staging/media/as102/as102_fw.c @@ -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; |