diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2016-07-19 15:24:26 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-08-24 08:41:59 -0400 |
commit | c6e4b8c1d3a81a2611e0a045f1c1e34ad3fb092e (patch) | |
tree | 3fd995d1716d655329850f1225d46a7e4271034b /drivers/media | |
parent | fe094dad25ec28585a591db823c768af2b2be1f4 (diff) |
[media] tw686x: 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@s-opensource.com>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/pci/tw686x/tw686x-video.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/pci/tw686x/tw686x-video.c b/drivers/media/pci/tw686x/tw686x-video.c index be257d0257a6..24a813256c99 100644 --- a/drivers/media/pci/tw686x/tw686x-video.c +++ b/drivers/media/pci/tw686x/tw686x-video.c | |||
@@ -1120,8 +1120,7 @@ void tw686x_video_free(struct tw686x_dev *dev) | |||
1120 | for (ch = 0; ch < max_channels(dev); ch++) { | 1120 | for (ch = 0; ch < max_channels(dev); ch++) { |
1121 | struct tw686x_video_channel *vc = &dev->video_channels[ch]; | 1121 | struct tw686x_video_channel *vc = &dev->video_channels[ch]; |
1122 | 1122 | ||
1123 | if (vc->device) | 1123 | video_unregister_device(vc->device); |
1124 | video_unregister_device(vc->device); | ||
1125 | 1124 | ||
1126 | if (dev->dma_ops->free) | 1125 | if (dev->dma_ops->free) |
1127 | for (pb = 0; pb < 2; pb++) | 1126 | for (pb = 0; pb < 2; pb++) |