diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2014-08-08 08:59:02 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-08-21 16:25:32 -0400 |
commit | 0821344d9e805912f1971c58318547ea62984321 (patch) | |
tree | a0897200f6ffc025246fc23a8524cbbabc69e1f1 /drivers/media/v4l2-core | |
parent | 83acb75d0dd37bd2eb24168f26d01c747ece3fc7 (diff) |
[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 <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Diffstat (limited to 'drivers/media/v4l2-core')
-rw-r--r-- | drivers/media/v4l2-core/videobuf2-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index eb86913349fc..5b808e25fc09 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c | |||
@@ -36,7 +36,7 @@ module_param(debug, int, 0644); | |||
36 | #define dprintk(level, fmt, arg...) \ | 36 | #define dprintk(level, fmt, arg...) \ |
37 | do { \ | 37 | do { \ |
38 | if (debug >= level) \ | 38 | if (debug >= level) \ |
39 | pr_debug("vb2: %s: " fmt, __func__, ## arg); \ | 39 | pr_info("vb2: %s: " fmt, __func__, ## arg); \ |
40 | } while (0) | 40 | } while (0) |
41 | 41 | ||
42 | #ifdef CONFIG_VIDEO_ADV_DEBUG | 42 | #ifdef CONFIG_VIDEO_ADV_DEBUG |