On Tue, Oct 21, 2008 at 11:36:12AM +0800, Bryan Wu wrote:
> +config SND_MULTICHAN_SUPPORT
> + bool "Enable Multichannel Support"
> + depends on SND_BF5XX_AC97
> + default n
> + help
> + Say y if you want AC97 driver to support up to 5.1 channel audio.
> + this mode will consume much more memory for DMA.
> +
Should this be a user visible option or selected by the machine drivers?
To be honest, I'm surprised that the multi-channel support can't only
incur the extra memory consumpton when playing back a multi-channel
stream - I'd have expected the hardware to work in the same way for a
normal stereo stream even if multi-channel support is built in.
> -void bf5xx_ac97_to_pcm(const struct ac97_frame *src, __u32 *dst, \
> +void bf5xx_ac97_to_pcm(const struct ac97_frame *src, __u16 *dst, \
> size_t count)
> {
Separately please submit a patch removing the needless continuation
character here.
> diff --git a/sound/soc/codecs/ad1980.c b/sound/soc/codecs/ad1980.c
> index 1397b8e..e22fbdf 100644
> --- a/sound/soc/codecs/ad1980.c
> +++ b/sound/soc/codecs/ad1980.c
The codec driver changes should be submitted as a separate patch.
They're good to go as-is.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
