diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-02 12:20:23 (GMT) |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-02-02 15:34:46 (GMT) |
| commit | a1408379ba9d87d66691835c7cdc2645793e311a (patch) | |
| tree | dd12f993e69c704bdcf22811797dd079de1b3895 | |
| parent | 47e604c5ddf128b5102cfd5d88cd5e2f243e1c2f (diff) | |
| download | linux-a1408379ba9d87d66691835c7cdc2645793e311a.tar.xz | |
[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.c | 4 |
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 4d5e2f4..5382506 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); |