media: cx231xx: Consolidate dmesg output
The memory allocations in cx231xx_init_*() happen all in task context with GFP_KERNEL. Therefore a dev_err() trying to deduce whether this is called from task or interrupt context is pretty useless. Remove these historical leftovers. 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>
This commit is contained in:
Родитель
994388f228
Коммит
8b77d1f99e
|
@ -1061,9 +1061,8 @@ int cx231xx_init_isoc(struct cx231xx *dev, int max_packets,
|
|||
&urb->transfer_dma);
|
||||
if (!dev->video_mode.isoc_ctl.transfer_buffer[i]) {
|
||||
dev_err(dev->dev,
|
||||
"unable to allocate %i bytes for transfer buffer %i%s\n",
|
||||
sb_size, i,
|
||||
in_interrupt() ? " while in int" : "");
|
||||
"unable to allocate %i bytes for transfer buffer %i\n",
|
||||
sb_size, i);
|
||||
cx231xx_uninit_isoc(dev);
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
@ -1197,9 +1196,8 @@ int cx231xx_init_bulk(struct cx231xx *dev, int max_packets,
|
|||
&urb->transfer_dma);
|
||||
if (!dev->video_mode.bulk_ctl.transfer_buffer[i]) {
|
||||
dev_err(dev->dev,
|
||||
"unable to allocate %i bytes for transfer buffer %i%s\n",
|
||||
sb_size, i,
|
||||
in_interrupt() ? " while in int" : "");
|
||||
"unable to allocate %i bytes for transfer buffer %i\n",
|
||||
sb_size, i);
|
||||
cx231xx_uninit_bulk(dev);
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
|
|
@ -408,9 +408,8 @@ int cx231xx_init_vbi_isoc(struct cx231xx *dev, int max_packets,
|
|||
kzalloc(sb_size, GFP_KERNEL);
|
||||
if (!dev->vbi_mode.bulk_ctl.transfer_buffer[i]) {
|
||||
dev_err(dev->dev,
|
||||
"unable to allocate %i bytes for transfer buffer %i%s\n",
|
||||
sb_size, i,
|
||||
in_interrupt() ? " while in int" : "");
|
||||
"unable to allocate %i bytes for transfer buffer %i\n",
|
||||
sb_size, i);
|
||||
cx231xx_uninit_vbi_isoc(dev);
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
|
Загрузка…
Ссылка в новой задаче