[alsa-devel] SigmaTel STAC9221 - Mux Amp-Out > 0x02 == microphone not working

[Takashi Iwai]

At Tue, 29 May 2007 17:49:54 +0100,
Ivan N. Zlatev wrote:
> 
> On 5/29/07, Takashi Iwai <tiwai at suse.de> wrote:
> > At Tue, 29 May 2007 17:29:10 +0200,
> > I wrote:
> > >
> > > At Tue, 29 May 2007 16:20:33 +0100,
> > > Ivan N. Zlatev wrote:
> > > >
> > > > On 5/29/07, Takashi Iwai <tiwai at suse.de> wrote:
> > > > >
> > > > > And, does this volume have any influence on the quality of recorded
> > > > > sound at all?  If not, we should remove this control rather than
> > > > > keeping a dangerous thing...
> > > > >
> > > >
> > > > Yes. This controls the microphone boost level (without boost the sound
> > > > recorded has *very* low volume). Also Windows shows  up a +20db
> > > > microphone boost slider on this machine.
> > >
> > > Ah, OK.  Then the control seems indicates the wrong max level indeed.
> > > So, we need two fixes:
> > >
> > > - rename "Mux Capture Volume" to more intuitive one, such as "Capture
> > >   Boost" or something like that.
> > >
> > > - limit the max level of this volume to 2.
> > >
> > > The latter is a bit hard to fix.  Let's see whether I can do some
> > > hack...
> >
> > Could you check the patch below?
> > (This doesn't rename but just fix the latter item.)
> >
> >
> > Takashi
> >
> > diff -r 200fc3a7ef62 pci/hda/hda_codec.c
> > --- a/pci/hda/hda_codec.c       Tue May 29 18:01:06 2007 +0200
> > +++ b/pci/hda/hda_codec.c       Tue May 29 18:17:15 2007 +0200
> > @@ -711,6 +711,19 @@ static u32 query_amp_caps(struct hda_cod
> >                         info->status |= INFO_AMP_CAPS;
> >         }
> >         return info->amp_caps;
> > +}
> > +
> > +int snd_hda_override_amp_caps(struct hda_codec *codec, hda_nid_t nid, int dir,
> > +                             unsigned int caps)
> > +{
> > +       struct hda_amp_info *info;
> > +
> > +       info = get_alloc_amp_hash(codec, HDA_HASH_KEY(nid, dir, 0));
> > +       if (!info)
> > +               return -EINVAL;
> > +       info->amp_caps = caps;
> > +       info->status |= INFO_AMP_CAPS;
> > +       return 0;
> >  }
> >
> >  /*
> > diff -r 200fc3a7ef62 pci/hda/hda_local.h
> > --- a/pci/hda/hda_local.h       Tue May 29 18:01:06 2007 +0200
> > +++ b/pci/hda/hda_local.h       Tue May 29 18:17:02 2007 +0200
> > @@ -277,5 +277,7 @@ static inline u32 get_wcaps(struct hda_c
> >         return codec->wcaps[nid - codec->start_nid];
> >  }
> >
> > +int snd_hda_override_amp_caps(struct hda_codec *codec, hda_nid_t nid, int dir,
> > +                             unsigned int caps);
> >
> >  #endif /* __SOUND_HDA_LOCAL_H */
> > diff -r 200fc3a7ef62 pci/hda/patch_sigmatel.c
> > --- a/pci/hda/patch_sigmatel.c  Tue May 29 18:01:06 2007 +0200
> > +++ b/pci/hda/patch_sigmatel.c  Tue May 29 18:12:30 2007 +0200
> > @@ -2033,6 +2033,12 @@ static int patch_stac925x(struct hda_cod
> >
> >         codec->patch_ops = stac92xx_patch_ops;
> >
> > +       /* Fix Mux capture level; max to 2 */
> > +       snd_hda_override_amp_caps(codec, 0x12, HDA_OUTPUT,
> > +                                 (0 << AC_AMPCAP_OFFSET_SHIFT) |
> > +                                 (2 << AC_AMPCAP_NUM_STEPS_SHIFT) |
> > +                                 (0x27 << AC_AMPCAP_STEP_SIZE_SHIFT) |
> > +                                 (0 << AC_AMPCAP_MUTE_SHIFT));
> >         return 0;
> >  }
> >
> >
> 
> Works, BUT the last hunk is at the wrong place. The
> snd_hda_override_amp_caps call should be in patch_stac922x (...) and
> not in patch_stac925x (...)

Aha, a typical error :)
I committed the fixed patch to ALSA HG tree now.
Could you check the HG tree later whether it really works?

Also, any other pending issues?


thanks,

Takashi