[alsa-devel] [PATCH 2/2] ALSA: AACI: fix recording bug

[Takashi Iwai]

At Sun, 29 Nov 2009 16:39:59 +0000,
Russell King - ARM Linux wrote:
> 
> pcm->r[1].slots is the double rate slot information, not the
> capture information.  For capture, 'pcm' will already be the
> capture ac97 pcm structure.
> 
> Signed-off-by: Russell King <rmk+kernel at arm.linux.org.uk>
> ---
>  sound/arm/aaci.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/sound/arm/aaci.c b/sound/arm/aaci.c
> index 1456fc7..eb715e7 100644
> --- a/sound/arm/aaci.c
> +++ b/sound/arm/aaci.c
> @@ -518,7 +518,7 @@ static int aaci_pcm_hw_params(struct snd_pcm_substream *substream,
>  	else
>  		err = snd_ac97_pcm_open(aacirun->pcm, params_rate(params),
>  					params_channels(params),
> -					aacirun->pcm->r[1].slots);
> +					aacirun->pcm->r[0].slots);

Then we can get rid of the whole if check and the else block here.
The arguments to be passed are identical for both playback and capture
directions.


thanks,

Takashi