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/media/common | |
| parent | media: v4l2-core: v4l2-i2c: convert to new API with ERRPTR (diff) | |
| download | wireguard-linux-639c0a5b0503fb57127fa8972208d43020a9bcf6.tar.xz wireguard-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/media/common')
| -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"); |