diff options
| author |   Thomas Gleixner <tglx@linutronix.de> | 2020-10-13 16:26:17 +0200 |
|---|---|---|
| committer |   Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-11-16 10:31:10 +0100 |
| commit | 9bb83e8fe252f15ad30430e80eb7f9da66cacbb9 (patch) | |
| tree | ca1f95dbced804e4a86cd4cb5bfb8891b7d82e0d /drivers/media/platform/fsl-viu.c | |
| parent | media: mtk-vcodec: add missing put_device() call in mtk_vcodec_release_enc_pm() (diff) | |
| download | linux-dev-9bb83e8fe252f15ad30430e80eb7f9da66cacbb9.tar.xz linux-dev-9bb83e8fe252f15ad30430e80eb7f9da66cacbb9.zip | |
media: Bulk remove BUG_ON(in_interrupt())
None of these BUG_ON()'s is justified. BUG_ON() should only be used when
there is really no way to survive.
If at all these could be replaced by lockdep_assert_preemption_enabled() to
cover all invalid caller context and not just those covered by
in_interrupt().
But all functions which are invoked from those places contain already debug
mechanisms to catch wrong context, so having these extra checks is not
having any advantage.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/platform/fsl-viu.c')
| -rw-r--r-- | drivers/media/platform/fsl-viu.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/media/platform/fsl-viu.c b/drivers/media/platform/fsl-viu.c index 4f2a0f992905..ff625301e777 100644 --- a/drivers/media/platform/fsl-viu.c +++ b/drivers/media/platform/fsl-viu.c @@ -381,8 +381,6 @@ static void free_buffer(struct videobuf_queue *vq, struct viu_buf *buf) struct videobuf_buffer *vb = &buf->vb; void *vaddr = NULL; - BUG_ON(in_interrupt()); - videobuf_waiton(vq, &buf->vb, 0, 0); if (vq->int_ops && vq->int_ops->vaddr) |