diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-06-12 05:42:00 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-06-28 07:54:03 -0400 |
commit | 7534122d60066a50859c422ced4636deb23760bf (patch) | |
tree | 2fadf5a40993fea317b2c7f31a002c78ae79de86 /drivers/media/pci/cx18 | |
parent | 34dbb848d5e47f0a16abeb40885f93dd43045ff1 (diff) |
media: cx18: remove redundant zero check on retval
The check for a zero retval is redundant as all paths that lead to
this point have set retval to an error return value that is non-zero.
Remove the redundant check.
Detected by CoverityScan, CID#102589 ("Logically dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/pci/cx18')
-rw-r--r-- | drivers/media/pci/cx18/cx18-driver.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/media/pci/cx18/cx18-driver.c b/drivers/media/pci/cx18/cx18-driver.c index 8f314ca320c7..0c389a3fb4e5 100644 --- a/drivers/media/pci/cx18/cx18-driver.c +++ b/drivers/media/pci/cx18/cx18-driver.c | |||
@@ -1134,8 +1134,6 @@ free_mem: | |||
1134 | free_workqueues: | 1134 | free_workqueues: |
1135 | destroy_workqueue(cx->in_work_queue); | 1135 | destroy_workqueue(cx->in_work_queue); |
1136 | err: | 1136 | err: |
1137 | if (retval == 0) | ||
1138 | retval = -ENODEV; | ||
1139 | CX18_ERR("Error %d on initialization\n", retval); | 1137 | CX18_ERR("Error %d on initialization\n", retval); |
1140 | 1138 | ||
1141 | v4l2_device_unregister(&cx->v4l2_dev); | 1139 | v4l2_device_unregister(&cx->v4l2_dev); |