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@xxxxxxxxxxxxxxxx > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > -- Tobin Davis > > "You've got to have a gimmick if your band sucks." > -- Gary Giddens > > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
