On 5/29/07, Takashi Iwai <tiwai@xxxxxxx> 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@xxxxxxx> 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 (...)
Cheers!
--
Ivan N. Zlatev
Web: http://www.i-nZ.net
"It's all some kind of whacked out conspiracy."
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
