diff options
author | Aditya Pakki <pakki001@umn.edu> | 2019-12-19 21:29:24 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-02-24 15:22:31 +0100 |
commit | 639c0a5b0503fb57127fa8972208d43020a9bcf6 (patch) | |
tree | 6ade4633ce2f2eebefbf7bd29d7bb58f4033cec2 /drivers | |
parent | 2e1b100cdceb41ad05edc467a43e5affd2c87b6d (diff) | |
download | linux-639c0a5b0503fb57127fa8972208d43020a9bcf6.tar.gz linux-639c0a5b0503fb57127fa8972208d43020a9bcf6.tar.bz2 linux-639c0a5b0503fb57127fa8972208d43020a9bcf6.zip |
media: media/saa7146: fix incorrect assertion in saa7146_buffer_finish
In saa7146_buffer_finish, the code for q->curr to be NULL is
already present and asserting for NULL is unnecessary. This patch
elimiates such a check.
Signed-off-by: Aditya Pakki <pakki001@umn.edu>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/media/common/saa7146/saa7146_fops.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/media/common/saa7146/saa7146_fops.c b/drivers/media/common/saa7146/saa7146_fops.c index aabb830e7468..d7e83b55ddca 100644 --- a/drivers/media/common/saa7146/saa7146_fops.c +++ b/drivers/media/common/saa7146/saa7146_fops.c @@ -97,8 +97,6 @@ void saa7146_buffer_finish(struct saa7146_dev *dev, DEB_EE("dev:%p, dmaq:%p, state:%d\n", dev, q, state); DEB_EE("q->curr:%p\n", q->curr); - BUG_ON(!q->curr); - /* finish current buffer */ if (NULL == q->curr) { DEB_D("aiii. no current buffer\n"); |