Re: [PATCH] drm/nouveau: add default debug level override

[Maarten Lankhorst <maarten.lankhorst-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>]

Hey,

Op 13-09-12 00:50, Marcin Slusarz schreef:
> Useful when we don't know which subsystem is failing.
>
> Signed-off-by: Marcin Slusarz <marcin.slusarz-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  drivers/gpu/drm/nouveau/core/core/option.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/nouveau/core/core/option.c b/drivers/gpu/drm/nouveau/core/core/option.c
> index 62a432e..fde06b0 100644
> --- a/drivers/gpu/drm/nouveau/core/core/option.c
> +++ b/drivers/gpu/drm/nouveau/core/core/option.c
> @@ -22,9 +22,16 @@
>   * Authors: Ben Skeggs
>   */
>  
> +#include <linux/module.h>
>  #include <core/option.h>
>  #include <core/debug.h>
>  
> +MODULE_PARM_DESC(debug_level, "nv_printk level (0-"
> +		__stringify(CONFIG_NOUVEAU_DEBUG) ", default "
> +		__stringify(CONFIG_NOUVEAU_DEBUG_DEFAULT) ")");
> +static int nouveau_debug_level = CONFIG_NOUVEAU_DEBUG_DEFAULT;
> +module_param_named(debug_level, nouveau_debug_level, int, 0400);
Why make default 0400? 0600 or 0644 would make more sense..

~Maarten