Planned changes

Planned changes

[Takashi Iwai]

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

Re: Planned changes

[Tobin Davis]

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.

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.

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@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

Re: Planned changes

[Takashi Iwai]

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@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                                                
> 
>