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@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; }