[media] vb2: use pr_info instead of pr_debug
authorHans Verkuil <[email protected]>
Fri, 8 Aug 2014 12:59:02 +0000 (09:59 -0300)
committerMauro Carvalho Chehab <[email protected]>
Thu, 21 Aug 2014 20:25:32 +0000 (15:25 -0500)
commit0821344d9e805912f1971c58318547ea62984321
treea0897200f6ffc025246fc23a8524cbbabc69e1f1
parent83acb75d0dd37bd2eb24168f26d01c747ece3fc7
[media] vb2: use pr_info instead of pr_debug

Modern kernels enable dynamic printk support, which is fine, except when it is
combined with a debug module option. Enabling debug in videobuf2-core now produces
no debugging unless it is also enabled through the dynamic printk support in debugfs.

Either use a debug module option + pr_info, or use pr_debug without a debug module
option. In this case the fact that you can set various debug levels is very useful,
so I believe that for videobuf2-core.c we should use pr_info.

The mix of the two is very confusing: I've spent too much time already trying to
figure out why I am not seeing any debug output in the kernel log when I do:

echo 1 >/sys/modules/videobuf2_core/parameters/debug

Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
drivers/media/v4l2-core/videobuf2-core.c