CONFIG_SND_DEBUG (was: Re: [alsa-devel] [PATCH] ALSA: get rid of CONFIG_SND_VERBOSE_PRINTK)

[Andy Whitcroft]

On Wed, Jun 05, 2013 at 09:47:30AM +0200, David Henningsson wrote:
> >>>On Wed, 2013-06-05 at 08:54 +0200, Takashi Iwai wrote:
> >>>>At Tue, 04 Jun 2013 23:52:01 -0700, Joe Perches wrote:
> >>>>>On Wed, 2013-06-05 at 08:32 +0200, Takashi Iwai wrote:
> >>>>>>Yes.  These are snd_printd() just to be conditionally built in.
> >>>>>>But in most cases it's rather useful to print them (as most distros
> >>>>>>set CONFIG_SND_DEBUG=y).
> >>>>>Ubuntu doesn't,  I believe Fedora doesn't.
> >>>>Then they should have done so :)
> >>>But they don't, so what distros do?
> >>
> >>RedHat (including Fedora) and SUSE do at least.
> >
> >Mandriva does too.  (still looking around for others)
> >
> >We can ask Ubuntu to enable CONFIG_SND_DEBUG.
> >(cc'd Andy Whitcroft, Leann Ogasawara and David Henningsson)
> >
> >Maybe there are others Canonical folk that
> >should be cc'd?
> 
> Adding kernel team mailing list to CC.
> 
> Andy/Leann - apparently CONFIG_SND_DEBUG is on by default from
> upstream, and we explicitly disable it. Is there any reason why we
> do that?

config SND_DEBUG
        bool "Debug"
        help
          Say Y here to enable ALSA debug code.

It is off by default in upstream, and the really helpful description
would cirtainly tend to lead to it being disabled.  But if it is helpful
to your debugging efforts David then I suspect we can enable it in Saucy
and see what happens.

-apw