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@xxxxxxxxxxxxxxxx> > --- > 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 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel