From: Aditya Pakki Date: Thu, 19 Dec 2019 20:29:24 +0000 (+0100) Subject: media: media/saa7146: fix incorrect assertion in saa7146_buffer_finish X-Git-Tag: v5.10.7~2993^2~231 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=639c0a5b0503fb57127fa8972208d43020a9bcf6;p=platform%2Fkernel%2Flinux-rpi.git 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 Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/common/saa7146/saa7146_fops.c b/drivers/media/common/saa7146/saa7146_fops.c index aabb830..d7e83b5 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");