diff options
| author | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-05-19 08:00:56 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-06-09 18:30:09 -0300 |
| commit | 09f2082e23ad8e89e10152b8701d3511ace45290 (patch) | |
| tree | 9d18d835e169f7e88eb9561d5fd11e94968f289b /drivers/media/usb/cx231xx/cx231xx-vbi.c | |
| parent | fa8cb6444c3236d2bad7460bdfdb2685f82b7ee4 (diff) | |
| download | linux-09f2082e23ad8e89e10152b8701d3511ace45290.tar.bz2 | |
[media] usb drivers: use BUG_ON() instead of if () BUG
Some USB drivers have a logic at the VB buffer handling like:
if (in_interrupt())
BUG();
Use, instead:
BUG_ON(in_interrupt());
Btw, this logic looks weird on my eyes. We should convert them
to use VB2, in order to avoid those crappy things.
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/usb/cx231xx/cx231xx-vbi.c')
| -rw-r--r-- | drivers/media/usb/cx231xx/cx231xx-vbi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/usb/cx231xx/cx231xx-vbi.c b/drivers/media/usb/cx231xx/cx231xx-vbi.c index 80261ac40208..a08014d20a5c 100644 --- a/drivers/media/usb/cx231xx/cx231xx-vbi.c +++ b/drivers/media/usb/cx231xx/cx231xx-vbi.c @@ -192,8 +192,7 @@ static void free_buffer(struct videobuf_queue *vq, struct cx231xx_buffer *buf) struct cx231xx_fh *fh = vq->priv_data; struct cx231xx *dev = fh->dev; unsigned long flags = 0; - if (in_interrupt()) - BUG(); + BUG_ON(in_interrupt()); /* We used to wait for the buffer to finish here, but this didn't work because, as we were keeping the state as VIDEOBUF_QUEUED, |