summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-02 09:20:23 -0300
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2015-02-02 13:34:46 -0200
commita1408379ba9d87d66691835c7cdc2645793e311a (patch)
treedd12f993e69c704bdcf22811797dd079de1b3895
parent47e604c5ddf128b5102cfd5d88cd5e2f243e1c2f (diff)
downloadlinux-a1408379ba9d87d66691835c7cdc2645793e311a.tar.gz
linux-a1408379ba9d87d66691835c7cdc2645793e311a.tar.bz2
linux-a1408379ba9d87d66691835c7cdc2645793e311a.zip
[media] staging: bcm2048: Delete an unnecessary check before the function call "video_unregister_device"
The video_unregister_device() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r--drivers/staging/media/bcm2048/radio-bcm2048.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c
index 4d5e2f400ff6..538250667918 100644
--- a/drivers/staging/media/bcm2048/radio-bcm2048.c
+++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
@@ -2684,9 +2684,7 @@ static int __exit bcm2048_i2c_driver_remove(struct i2c_client *client)
vd = bdev->videodev;
bcm2048_sysfs_unregister_properties(bdev, ARRAY_SIZE(attrs));
-
- if (vd)
- video_unregister_device(vd);
+ video_unregister_device(vd);
if (bdev->power_state)
bcm2048_set_power_state(bdev, BCM2048_POWER_OFF);