[alsa-devel] i think ac97 needs an is_surback_on() function

[Takashi Iwai]

At Fri, 22 Aug 2008 15:20:09 -0700,
John L. Utz III wrote:
> 
> ac97_patch.c provides two helper functions that wrap the channel_mode  
> variable, but they only take you out to 5.1 Audio:
> 
> static inline int is_surround_on(struct snd_ac97 *ac97)
> {
> 	return ac97->channel_mode >= 1;
> }
> 
> static inline int is_clfe_on(struct snd_ac97 *ac97)
> {
> 	return ac97->channel_mode >= 2;
> }
> 
> 
> For 7.1 Audio, i think there needs to be a test for 'Surround Back' or  
> 'Back Surround'
> 
> static inline int is_surback_on(struct snd_ac97 *ac97)
> {
> 	return ac97->channel_mode >= 4;
> }
> 
> 
> Am i correct? or am i mistaken? agreement or corrections appreciated!

It's correct.  The code for 7.1-audio capability was added recently.


Takashi