[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