[alsa-devel] Planned changes

Takashi Iwai tiwai at suse.de
Fri May 16 18:46:19 CEST 2008


At Fri, 16 May 2008 09:35:59 -0700,
Tobin Davis wrote:
> 
> On the debug side, I think it would be nicer if we could just compile with all
> debugging available, then specify a debug=<level> option on module load or
> through /sys writes (echo 3 >/sys/module/snd_hda-intel/parameters/debug).  This
> way, each module can be debugged independently without a lot of debug messages
> from other modules.

I don't like the option of each module.  Then you'll have to add
option to each driver code.  Rather a global dynamic debug-level
variable would be easier to use and implement.

> As to the Kconfig option, switching to that would then require users/developers
> to edit a kernel config file.  Being able to select which module to compile is
> far easier for people that only care about one or two specific cards.

You can choose the kconfig via --with-kconfig configure option, such
as,
	% ./configure --with-kconfig=hda-intel,usb-audio

The fact is that --with-kconfig supersedes the --with-cards and
--with-card-options options.


Takashi

> 
> Just my $.02(USD) worth.
> 
> Tobin
> 
> On Fri, 2008-05-16 at 16:36 +0200, Takashi Iwai wrote:
> 
>     Hi,
>     
>     there are some patches floating around on my experimental tree, which
>     I'd like to merge to the ALSA upstream.  At least, the following two
>     are ready for merge:
>     
>     - Use menuconfig in ALSA kconfig
>     - Convert CONFIG_SND_DEBUG_DETECT to CONFIG_SND_DEBUG_VERBOSE
>     
>     The first one is to change from menu to menuconfig in ALSA Kconfig
>     files.  There have been some patches by others, but they haven't been
>     applied because of the problem with alsa-driver build.  Now I hacked
>     alsa-driver build again so that it works with menuconfig somehow.
>     
>     The drawback is that it changes the configure option.  Instead of
>     --with-cards and --with-card-options, now we have a single
>     --with-kconfigs.  I think, however, it's rather an improvement.
>     
>     The next one, renaming to CONFIG_SND_DEBUG_VERBOSE, is easy.
>     The reason is that CONFIG_SND_DEBUG_DETECT and snd_printdd() are used
>     almostly exclusively for verbose debug messages.  This may reduce some
>     misunderstanding about this option.
>     
>     The patches can be found on my experimental tree:
>         git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-unstable-2.6.git
>         git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/alsa-driver-build-unstable.git
>     
>     If there is no objection, I'll apply the above two changes in the next
>     week.
> 
>     thanks,
>     
>     Takashi
>     _______________________________________________
>     Alsa-devel mailing list
>     Alsa-devel at alsa-project.org
>     http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>     
> -- Tobin Davis                                                                 
> 
> "You've got to have a gimmick if your band sucks."                             
>                 -- Gary Giddens                                                
> 
> 


More information about the Alsa-devel mailing list