- Alsa-devel - mailman.alsa-project.org

[alsa-devel] Applying Herton supplied patches ...
by Guy Stalnaker 14 Feb '09

14 Feb '09

herton or Takashi, How does one go about applying a patch like the ones attached to Message 6? And to which files are they applied? I assume they are meant for the alsa-driver code, but that's as much as I know (well, I know how to run ./configure, make, and make sudo :). I'd very much like to see if the patches work for my HP dv4 1225. Regards, Guy > Message: 6 > Date: Sat, 14 Feb 2009 17:13:55 -0200 > From: Herton Ronaldo Krzesinski <herton(a)mandriva.com.br> > Subject: Re: [alsa-devel] [Fwd: HP DV4 1155 SE - No Sound through > speakers...] > To: alsa-devel(a)alsa-project.org > Cc: Takashi Iwai <tiwai(a)suse.de>, Tim Barnette > <tim.barnette(a)gmail.com>, Jean-Pierre Andr? > <jean-pierre.andre(a)wanadoo.fr> > Message-ID: <200902141713.55843.herton(a)mandriva.com.br> > Content-Type: text/plain; charset="iso-8859-1" > > On Saturday 14 February 2009 08:24:15 Takashi Iwai wrote: >> > At Fri, 13 Feb 2009 14:22:00 -0500, >> > Tim Barnette wrote: >>> > > >>> > > ..And it worked! I now have sound! Can you tell me what I actually did here? >> > >> > The problem was that your BIOS didn't set the correct default pin >> > configuration. The driver doesn't know which pin should be used for >> > the speaker output, as a consequence, it's set as INPUT. >> > Now you set it OUTPUT manually by hda-verb, and (fortunately) the >> > route to that pin was alive even though it wasn't declared, you get >> > the speaker output now. >> > >> > The next step is to implement a proper pin-config to override by >> > the driver. This won't be much work, but hey, wait for a while. >> > >> > (Or, did Herton already work on that?) > > I had added before pin configs for which source came from *.ini files of IDT > windows drivers provided by HP. But I don't know if they are at all necessary, > may be just setting config of 0x0d pin should suffice for all hp-dv5 models, > as all of them have speaker on 0x0d from all reports and also looking at the > *.ini files. > > I have three patches, you can choose one of them :) , see attached. The first > one is my previous pin configs work "let's set for all cases the pin configs > based on *.ini files", the others are same smaller change/less intrusive that > instead just deals with this hp dv4 case. > >> > >> > >> > thanks, >> > >> > Takashi >> > >>> > > >>> > > On Fri, Feb 13, 2009 at 2:16 PM, Tim Barnette <tim.barnette(a)gmail.com> wrote: >>> > > >>> > > Takashi, I was able to build hda-verb and when I executed the command >>> > > below, it responded with: >>> > > nid=0xd, verb=0x707, param=0x40 >>> > > value=0x0 >>> > > >>> > > On Mon, Feb 2, 2009 at 10:25 AM, Takashi Iwai <tiwai(a)suse.de> wrote: >>> > > >>> > > At Mon, 2 Feb 2009 10:17:03 -0500, >>> > > Tim Barnette wrote: >>> > > > >>> > > > Thanks, Takashi. I made this change and rebooted. >>> > > > >>> > > > I noticed right away the mute light is no longer on... but still no >>> > > sound >>> > > > through speakers, only headphones.. >>> > > > >>> > > > /sys/module/snd_hda_intel/parameters/model >>> > > > contains >>> > > > hp-dv5,<NULL>,<NULL>,<NULL>,<NULL>,<NULL>,<NULL>,<NULL> >>> > > > >>> > > > attached is the alsa-info.txt you requested.. >>> > > >>> > > Did you try the latest alsa-driver snapshot? >>> > > >>> > > If it still doesn't work, I guess this is a BIOS problem. >>> > > The other HP dv laptops show the widget 0x0d as the speaker while >>> > > yours is unused. >>> > > >>> > > Try to run: >>> > > hda-verb /dev/snd/hwC0D0 0x0d SET_PIN_WID 0x40 >>> > > >>> > > Takashi > > -- > []'s > Herton >

2 1

[alsa-devel] [PATCH 0/3] ASoC Blackfin updates
by Bryan Wu 14 Feb '09

14 Feb '09

Hi Mark, Please review these patches. Is it possible to add them to 2.6.30 queue? Thanks -Bryan

2 8

Re: [alsa-devel] [PATCH 0/4] Intel HDMI audio fixes
by Takashi Iwai 14 Feb '09

14 Feb '09

At Fri, 13 Feb 2009 09:28:22 -0800, Shane W wrote: > > On Thu, Feb 12, 2009 at 12:41:11PM +0800, Wu Fengguang wrote: > > static const char *channel_name[MAX_CHANNELS] = { > > /* 0 */ N_("Front Left"), > > /* 1 */ N_("Front Right"), > > /* 2 */ N_("Rear Left"), > > /* 3 */ N_("Rear Right"), > > /* 4 */ N_("Center"), /* wav file is "Front Center.wav" */ > > /* 5 */ N_("LFE"), /* wav file is "Rear Center.wav" */ > > /* 6 */ N_("Side Left"), > > /* 7 */ N_("Side Right"), > > > > Whereas the HDMI channel mapping selected for 6-channel playback is > > > > { .ca_index = 0x0b, .speakers = { 0, 0, RR, RL, FC, LFE, FR, FL } }, > > > > Here the two mappings disagree on channels other than Front Left/Right. > > > > In theory the channel mapping should be adjusted via AC_VERB_SET_HDMI_CHAN_SLOT. > > However I found that verb takes no effect for G35/G45. So I'm afraid > > there's no trivial ways we can adjust the channel mapping. > > If the driver can't reorder the channels to what ALSA > needs, is it possible to do it somewhere else like > HDA-Intel.conf and use the routing plugin somehow to > reorder things. Say for plughw:0,3 or modify the > surround51 device? > > I don't know much about this interface but it looks like > channel routing is possible here. Well, it's a good question. Right now, surround51 is used for the analog output which usually has the "right" ALSA channel mapping. So, we need to detect and switch the mapping using a plugin on the fly, and that's not so trivial. Besides, the problem arises when both analog 5.1 and HDMI 5.1 are implemented in the same "card". Since a "card" is defined by a controller, not by a codec as now, one card may have both 5.1 outputs. We might need to switch to a codec-based card approach sometime later, but I've postponed it simply because it can break so easily the compatibility with older versions... Anyway, the channel mapping issue is a missing piece. There are a few more issues, some to be implemented and some to be improved and fixed. Takashi

1 0

[alsa-devel] [PATCH]: ARM DaVinci ASoC: Fix module unload error
by chaithrika＠ti.com 13 Feb '09

13 Feb '09

From: Chaithrika U S <chaithrika(a)ti.com> Fix for the error when the audio module is unloaded. On unregistering the platform_device, platform_device_release will free the platform data.If platform data is static the kernel panics when it is freed. Instead use the platform device helper function to add data. This change has been tested on DM644x EVM. Signed-off-by: Chaithrika U S <chaithrika(a)ti.com> --- sound/soc/davinci/davinci-evm.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c index 1aad262..95c44bf 100644 --- a/sound/soc/davinci/davinci-evm.c +++ b/sound/soc/davinci/davinci-evm.c @@ -225,7 +225,7 @@ static int __init evm_init(void) platform_set_drvdata(evm_snd_device, &evm_snd_devdata); evm_snd_devdata.dev = &evm_snd_device->dev; - evm_snd_device->dev.platform_data = data; + platform_device_add_data(evm_snd_device, data, sizeof(data)); ret = platform_device_add_resources(evm_snd_device, resources, 1); if (ret) { -- 1.5.6

4 6

[alsa-devel] [PATCH] ALSA: hda - Consider additional capture source/selector in ALC889
by Herton Ronaldo Krzesinski 13 Feb '09

13 Feb '09

Currently code for capture source support in ALC889 only considers capture mixers. This change adds aditional support for ADC+selector present in ALC889, taking into account also the presence of an additional DMIC connection item in the selector. Signed-off-by: Herton Ronaldo Krzesinski <herton(a)mandriva.com.br> --- sound/pci/hda/patch_realtek.c | 105 ++++++++++++++++++++++++++++++----------- 1 files changed, 77 insertions(+), 28 deletions(-) Note: I did this patch without hardware to test on, just based on alsa-info.txt from one machine with ALC889 plus the ALC889 datasheet available from Realtek website, so this is just build tested. diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 863ab95..d81cb5e 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -238,6 +238,13 @@ enum { ALC883_MODEL_LAST, }; +/* styles of capture selection */ +enum { + CAPT_MUX = 0, /* only mux based */ + CAPT_MIX, /* only mixer based */ + CAPT_1MUX_MIX, /* first mux and other mixers */ +}; + /* for GPIO Poll */ #define GPIO_MASK 0x03 @@ -276,7 +283,7 @@ struct alc_spec { hda_nid_t *adc_nids; hda_nid_t *capsrc_nids; hda_nid_t dig_in_nid; /* digital-in NID; optional */ - unsigned char is_mix_capture; /* matrix-style capture (non-mux) */ + int capture_style; /* capture style (CAPT_*) */ /* capture source */ unsigned int num_mux_defs; @@ -294,7 +301,7 @@ struct alc_spec { /* dynamic controls, init_verbs and input_mux */ struct auto_pin_cfg autocfg; struct snd_array kctls; - struct hda_input_mux private_imux; + struct hda_input_mux private_imux[3]; hda_nid_t private_dac_nids[AUTO_CFG_MAX_OUTS]; /* hooks */ @@ -396,7 +403,8 @@ static int alc_mux_enum_put(struct snd_kcontrol *kcontrol, mux_idx = adc_idx >= spec->num_mux_defs ? 0 : adc_idx; imux = &spec->input_mux[mux_idx]; - if (spec->is_mix_capture) { + if (spec->capture_style && + !(spec->capture_style == CAPT_1MUX_MIX && !adc_idx)) { /* Matrix-mixer style (e.g. ALC882) */ unsigned int *cur_val = &spec->cur_mux[adc_idx]; unsigned int i, idx; @@ -4130,7 +4138,7 @@ static int new_analog_input(struct alc_spec *spec, hda_nid_t pin, static int alc880_auto_create_analog_input_ctls(struct alc_spec *spec, const struct auto_pin_cfg *cfg) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; int i, err, idx; for (i = 0; i < AUTO_PIN_LAST; i++) { @@ -4279,7 +4287,7 @@ static int alc880_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc880_volume_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; store_pin_configs(codec); return 1; @@ -5487,7 +5495,7 @@ static int alc260_auto_create_multi_out_ctls(struct alc_spec *spec, static int alc260_auto_create_analog_input_ctls(struct alc_spec *spec, const struct auto_pin_cfg *cfg) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; int i, err, idx; for (i = 0; i < AUTO_PIN_LAST; i++) { @@ -5647,7 +5655,7 @@ static int alc260_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc260_volume_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; store_pin_configs(codec); return 1; @@ -7087,7 +7095,7 @@ static int patch_alc882(struct hda_codec *codec) spec->stream_digital_playback = &alc882_pcm_digital_playback; spec->stream_digital_capture = &alc882_pcm_digital_capture; - spec->is_mix_capture = 1; /* matrix-style capture */ + spec->capture_style = CAPT_MIX; /* matrix-style capture */ if (!spec->adc_nids && spec->input_mux) { /* check whether NID 0x07 is valid */ unsigned int wcap = get_wcaps(codec, 0x07); @@ -7155,10 +7163,14 @@ static hda_nid_t alc883_adc_nids_rev[2] = { 0x09, 0x08 }; +#define alc889_adc_nids alc880_adc_nids + static hda_nid_t alc883_capsrc_nids[2] = { 0x23, 0x22 }; static hda_nid_t alc883_capsrc_nids_rev[2] = { 0x22, 0x23 }; +#define alc889_capsrc_nids alc882_capsrc_nids + /* input MUX */ /* FIXME: should be a matrix-type input source selection */ @@ -8977,6 +8989,8 @@ static int alc883_parse_auto_config(struct hda_codec *codec) { struct alc_spec *spec = codec->spec; int err = alc880_parse_auto_config(codec); + struct auto_pin_cfg *cfg = &spec->autocfg; + int i; if (err < 0) return err; @@ -8990,6 +9004,26 @@ static int alc883_parse_auto_config(struct hda_codec *codec) /* hack - override the init verbs */ spec->init_verbs[0] = alc883_auto_init_verbs; + /* setup input_mux for ALC889 */ + if (codec->vendor_id == 0x10ec0889) { + /* digital-mic input pin is excluded in alc880_auto_create..() + * because it's under 0x18 + */ + if (cfg->input_pins[AUTO_PIN_MIC] == 0x12 || + cfg->input_pins[AUTO_PIN_FRONT_MIC] == 0x12) { + struct hda_input_mux *imux = &spec->private_imux[0]; + for (i = 1; i < 3; i++) + memcpy(&spec->private_imux[i], + &spec->private_imux[0], + sizeof(spec->private_imux[0])); + imux->items[imux->num_items].label = "Int DMic"; + imux->items[imux->num_items].index = 0x0b; + imux->num_items++; + spec->num_mux_defs = 3; + spec->input_mux = spec->private_imux; + } + } + return 1; /* config found */ } @@ -9053,14 +9087,36 @@ static int patch_alc883(struct hda_codec *codec) spec->stream_name_analog = "ALC888 Analog"; spec->stream_name_digital = "ALC888 Digital"; } + if (!spec->num_adc_nids) { + spec->num_adc_nids = ARRAY_SIZE(alc883_adc_nids); + spec->adc_nids = alc883_adc_nids; + } + if (!spec->capsrc_nids) + spec->capsrc_nids = alc883_capsrc_nids; + spec->capture_style = CAPT_MIX; /* matrix-style capture */ break; case 0x10ec0889: spec->stream_name_analog = "ALC889 Analog"; spec->stream_name_digital = "ALC889 Digital"; + if (!spec->num_adc_nids) { + spec->num_adc_nids = ARRAY_SIZE(alc889_adc_nids); + spec->adc_nids = alc889_adc_nids; + } + if (!spec->capsrc_nids) + spec->capsrc_nids = alc889_capsrc_nids; + spec->capture_style = CAPT_1MUX_MIX; /* 1mux/Nmix-style + capture */ break; default: spec->stream_name_analog = "ALC883 Analog"; spec->stream_name_digital = "ALC883 Digital"; + if (!spec->num_adc_nids) { + spec->num_adc_nids = ARRAY_SIZE(alc883_adc_nids); + spec->adc_nids = alc883_adc_nids; + } + if (!spec->capsrc_nids) + spec->capsrc_nids = alc883_capsrc_nids; + spec->capture_style = CAPT_MIX; /* matrix-style capture */ break; } @@ -9071,13 +9127,6 @@ static int patch_alc883(struct hda_codec *codec) spec->stream_digital_playback = &alc883_pcm_digital_playback; spec->stream_digital_capture = &alc883_pcm_digital_capture; - if (!spec->num_adc_nids) { - spec->num_adc_nids = ARRAY_SIZE(alc883_adc_nids); - spec->adc_nids = alc883_adc_nids; - } - if (!spec->capsrc_nids) - spec->capsrc_nids = alc883_capsrc_nids; - spec->is_mix_capture = 1; /* matrix-style capture */ if (!spec->cap_mixer) set_capture_mixer(spec); @@ -10512,7 +10561,7 @@ static int alc262_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc262_volume_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; err = alc_auto_add_mic_boost(codec); if (err < 0) @@ -10881,7 +10930,7 @@ static int patch_alc262(struct hda_codec *codec) spec->stream_digital_playback = &alc262_pcm_digital_playback; spec->stream_digital_capture = &alc262_pcm_digital_capture; - spec->is_mix_capture = 1; + spec->capture_style = CAPT_MIX; if (!spec->adc_nids && spec->input_mux) { /* check whether NID 0x07 is valid */ unsigned int wcap = get_wcaps(codec, 0x07); @@ -11539,7 +11588,7 @@ static int alc268_auto_create_multi_out_ctls(struct alc_spec *spec, static int alc268_auto_create_analog_input_ctls(struct alc_spec *spec, const struct auto_pin_cfg *cfg) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; int i, idx1; for (i = 0; i < AUTO_PIN_LAST; i++) { @@ -11657,7 +11706,7 @@ static int alc268_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc268_volume_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; err = alc_auto_add_mic_boost(codec); if (err < 0) @@ -12511,7 +12560,7 @@ static int alc269_auto_create_analog_input_ctls(struct alc_spec *spec, */ if (cfg->input_pins[AUTO_PIN_MIC] == 0x12 || cfg->input_pins[AUTO_PIN_FRONT_MIC] == 0x12) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; imux->items[imux->num_items].label = "Int Mic"; imux->items[imux->num_items].index = 0x05; imux->num_items++; @@ -12567,7 +12616,7 @@ static int alc269_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc269_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; /* set default input source */ snd_hda_codec_write_cache(codec, alc269_capsrc_nids[0], 0, AC_VERB_SET_CONNECT_SEL, @@ -13483,7 +13532,7 @@ static int alc861_auto_create_hp_ctls(struct alc_spec *spec, hda_nid_t pin) static int alc861_auto_create_analog_input_ctls(struct alc_spec *spec, const struct auto_pin_cfg *cfg) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; int i, err, idx, idx1; for (i = 0; i < AUTO_PIN_LAST; i++) { @@ -13620,7 +13669,7 @@ static int alc861_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc861_auto_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; spec->adc_nids = alc861_adc_nids; spec->num_adc_nids = ARRAY_SIZE(alc861_adc_nids); @@ -14724,7 +14773,7 @@ static int alc861vd_parse_auto_config(struct hda_codec *codec) add_verb(spec, alc861vd_volume_init_verbs); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; err = alc_auto_add_mic_boost(codec); if (err < 0) @@ -14803,7 +14852,7 @@ static int patch_alc861vd(struct hda_codec *codec) spec->adc_nids = alc861vd_adc_nids; spec->num_adc_nids = ARRAY_SIZE(alc861vd_adc_nids); spec->capsrc_nids = alc861vd_capsrc_nids; - spec->is_mix_capture = 1; + spec->capture_style = CAPT_MIX; set_capture_mixer(spec); @@ -16397,7 +16446,7 @@ static int alc662_auto_create_extra_out(struct alc_spec *spec, hda_nid_t pin, static int alc662_auto_create_analog_input_ctls(struct alc_spec *spec, const struct auto_pin_cfg *cfg) { - struct hda_input_mux *imux = &spec->private_imux; + struct hda_input_mux *imux = &spec->private_imux[0]; int i, err, idx; for (i = 0; i < AUTO_PIN_LAST; i++) { @@ -16528,7 +16577,7 @@ static int alc662_parse_auto_config(struct hda_codec *codec) add_mixer(spec, spec->kctls.list); spec->num_mux_defs = 1; - spec->input_mux = &spec->private_imux; + spec->input_mux = &spec->private_imux[0]; add_verb(spec, alc662_auto_init_verbs); if (codec->vendor_id == 0x10ec0663) @@ -16613,7 +16662,7 @@ static int patch_alc662(struct hda_codec *codec) spec->adc_nids = alc662_adc_nids; spec->num_adc_nids = ARRAY_SIZE(alc662_adc_nids); spec->capsrc_nids = alc662_capsrc_nids; - spec->is_mix_capture = 1; + spec->capture_style = CAPT_MIX; if (!spec->cap_mixer) set_capture_mixer(spec); -- 1.6.1

3 5

[alsa-devel] ASoC update for 2.6.29
by Mark Brown 13 Feb '09

13 Feb '09

The following changes since commit 272edb00493af32c609f43bdf1d75141756fd999: Lopez Cruz, Misael (1): ASoC: Update SDP3430 machine driver for snd_soc_card are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound-2.6.git for-2.6.29 Mark Brown (1): ASoC: Only register AC97 bus if it's not done already sound/soc/soc-core.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-)

2 2

[alsa-devel] [PATCH] ASoC: Only register AC97 bus if it's not done already
by Mark Brown 13 Feb '09

13 Feb '09

ASoC supports both explicit codec drivers for AC97 devices and a simple driver which uses the standard ALSA AC97 framework for codec support. When used with the generic AC97 codec support that will provide the ad hoc AC97 device for drivers like touchscreens to attach to so the core shouldn't do so. Signed-off-by: Mark Brown <broonie(a)opensource.wolfsonmicro.com> --- sound/soc/soc-core.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 55fdb4a..ec3f8bb 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -1385,7 +1385,10 @@ int snd_soc_init_card(struct snd_soc_device *socdev) mutex_lock(&codec->mutex); #ifdef CONFIG_SND_SOC_AC97_BUS - if (ac97) { + /* Only instantiate AC97 if not already done by the adaptor + * for the generic AC97 subsystem. + */ + if (ac97 && strcmp(codec->name, "AC97") != 0) { ret = soc_ac97_dev_register(codec); if (ret < 0) { printk(KERN_ERR "asoc: AC97 device register failed\n"); -- 1.5.6.5

1 0

Re: [alsa-devel] [PATCH] ALSA: hda - Consider additional capture source/selector in ALC889
by Takashi Iwai 13 Feb '09

13 Feb '09

At Fri, 13 Feb 2009 13:29:38 +0100, Mauro Condarelli wrote: > > Takashi Iwai wrote: > > At Fri, 13 Feb 2009 11:10:10 +0000 (UTC), > > Mauro Condarelli wrote: > > > >> Hi, > >> sorry to disturb. > >> I have an acer Aspire 8930 and I am fighting with sound. > >> I have an up-to-date ubuntu Intrepid distribution. > >> I had NO sound at all. > >> I followed instructions upgrading to latest alsa-driver-1.0.19 added "options > >> snd-hda-intel model=acer" to /etc/modprobe.d/alsa-base and > >> "/usr/local/bin/hda-verb /dev/snd/hwC0D0 0x15 SET_EAPD_BTLENABLE 2" to > >> /etc/rc.local. > >> JOY! I have sound. > >> So far so good. > >> Now my problem is I do not have capture (intrnal or external mic are not > >> working); I kept searching and found this thread. > >> I downloaded the alsa-driver-snapshot-tiwai from ftp.kernel.org, applied the > >> last patch (it seemed not applied in the snapshot) which went in with some line > >> discrepancies. > >> Removed the above settings and insatalled the new driver. > >> Result is now I have a *very* low volume audio and I'm unsure the mic is working > >> at all. > >> I'm obviously missing something. > >> Can someone be so kind to point me in the right direction, please? > >> I'm (obviously!) ready to offer all info required. I simply didn't want to make > >> this post any longer. > >> > > > > Try the latest snapshot below with model=auto option. > > ftp://ftp.kernel.org/pub/linux/kernel/people/tiwai/snapshot/alsa-driver-sna… > > > > If this doesn't work, please run alsa-info.sh with --no-upload option > > and attach the generated file. The script is found at > > http://www.alsa-project.org/alsa-info.sh > > > > > Hi Takashi, > many thanks for the really fast answer! > > > I was already using that snapshot (albeit with further patches). > I recompiled and reinstalled a fresh copy (to be on the safe side), but > the results are the same: *very* low volume even with all mixers up. And it's louder if you pass the model option? If so, please run alsa-info.sh on both states to compare. thanks, Takashi

1 0

[alsa-devel] [PATCH] snd-hda-intel: add quirk for Lenovo X200 laptop dock
by Aristeu Sergio Rozanski Filho 13 Feb '09

13 Feb '09

Currently the HP connector on X200 dock doesn't detect when a HP is connected nor allows sound to be played using it. This patch fixes the problem by adding a quirk for this specific model. It's possible that others have the same NID (0x19) to report when dock HP is connected, but I don't have access to any. Please Cc me in the reply since I'm not subscribed to alsa-devel@. Signed-off-by: Aristeu Rozanski <aris(a)redhat.com> --- sound/pci/hda/patch_conexant.c | 39 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+) --- a/sound/pci/hda/patch_conexant.c 2009-02-12 17:03:19.000000000 -0500 +++ b/sound/pci/hda/patch_conexant.c 2009-02-12 17:33:20.000000000 -0500 @@ -1806,6 +1806,40 @@ static struct hda_verb cxt5051_init_verb { } /* end */ }; +static struct hda_verb cxt5051_lenovo_x200_init_verbs[] = { + /* Line in, Mic */ + {0x17, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(0) | 0x03}, + {0x17, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_VREF80}, + {0x18, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(0) | 0x03}, + {0x18, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_VREF80}, + {0x1d, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_IN}, + {0x1d, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(0) | 0x03}, + /* SPK */ + {0x1a, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT}, + {0x1a, AC_VERB_SET_CONNECT_SEL, 0x00}, + /* HP, Amp */ + {0x16, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_HP}, + {0x16, AC_VERB_SET_CONNECT_SEL, 0x00}, + /* Docking HP */ + {0x19, AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_HP}, + {0x19, AC_VERB_SET_CONNECT_SEL, 0x00}, + /* DAC1 */ + {0x10, AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_UNMUTE}, + /* Record selector: Int mic */ + {0x14, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(0) | 0x44}, + {0x14, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(1) | 0x44}, + {0x15, AC_VERB_SET_AMP_GAIN_MUTE, AMP_IN_UNMUTE(0) | 0x44}, + /* SPDIF route: PCM */ + {0x1c, AC_VERB_SET_CONNECT_SEL, 0x0}, + /* EAPD */ + {0x1a, AC_VERB_SET_EAPD_BTLENABLE, 0x2}, /* default on */ + {0x16, AC_VERB_SET_UNSOLICITED_ENABLE, AC_USRSP_EN|CONEXANT_HP_EVENT}, + {0x17, AC_VERB_SET_UNSOLICITED_ENABLE, AC_USRSP_EN|CXT5051_PORTB_EVENT}, + {0x18, AC_VERB_SET_UNSOLICITED_ENABLE, AC_USRSP_EN|CXT5051_PORTC_EVENT}, + {0x19, AC_VERB_SET_UNSOLICITED_ENABLE, AC_USRSP_EN|CONEXANT_HP_EVENT}, + { } /* end */ +}; + /* initialize jack-sensing, too */ static int cxt5051_init(struct hda_codec *codec) { @@ -1823,18 +1857,21 @@ static int cxt5051_init(struct hda_codec enum { CXT5051_LAPTOP, /* Laptops w/ EAPD support */ CXT5051_HP, /* no docking */ + CXT5051_LENOVO_X200, /* Lenovo X200 laptop */ CXT5051_MODELS }; static const char *cxt5051_models[CXT5051_MODELS] = { [CXT5051_LAPTOP] = "laptop", [CXT5051_HP] = "hp", + [CXT5051_LENOVO_X200] = "Lenovo X200", }; static struct snd_pci_quirk cxt5051_cfg_tbl[] = { SND_PCI_QUIRK(0x14f1, 0x0101, "Conexant Reference board", CXT5051_LAPTOP), SND_PCI_QUIRK(0x14f1, 0x5051, "HP Spartan 1.1", CXT5051_HP), + SND_PCI_QUIRK(0x17aa, 0x20f2, "Lenovo X200", CXT5051_LENOVO_X200), {} }; @@ -1875,6 +1912,8 @@ static int patch_cxt5051(struct hda_code codec->patch_ops.unsol_event = cxt5051_hp_unsol_event; spec->mixers[0] = cxt5051_hp_mixers; break; + case CXT5051_LENOVO_X200: + spec->init_verbs[0] = cxt5051_lenovo_x200_init_verbs; default: case CXT5051_LAPTOP: codec->patch_ops.unsol_event = cxt5051_hp_unsol_event;

2 1

[alsa-devel] [PATCH] ALSA: hda - Change HP dv7 (103c:30f4) quirk from hp-m4 to hp-dv5 model
by Herton Ronaldo Krzesinski 13 Feb '09

13 Feb '09

Change HP dv7 quirk: although reported to work with hp-m4 model (https://bugzilla.novell.com/show_bug.cgi?id=445321), the original report doesn't contain info about testing of internal microphone. Recently I received a report about internal mic not working (https://qa.mandriva.com/show_bug.cgi?id=44855#c193), this must be related with the forced line in on pin 0x0e done with hp-m4 model. Thus change the current quirk from STAC_HP_M4 to STAC_HP_DV5, later reported to be fixed on a provided kernel with this change (https://qa.mandriva.com/show_bug.cgi?id=44855#c196). Signed-off-by: Herton Ronaldo Krzesinski <herton(a)mandriva.com.br> --- sound/pci/hda/patch_sigmatel.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.c index 3c84817..efc1aff 100644 --- a/sound/pci/hda/patch_sigmatel.c +++ b/sound/pci/hda/patch_sigmatel.c @@ -1811,7 +1811,7 @@ static struct snd_pci_quirk stac92hd71bxx_cfg_tbl[] = { SND_PCI_QUIRK(PCI_VENDOR_ID_HP, 0x30f2, "HP dv5", STAC_HP_M4), SND_PCI_QUIRK(PCI_VENDOR_ID_HP, 0x30f4, - "HP dv7", STAC_HP_M4), + "HP dv7", STAC_HP_DV5), SND_PCI_QUIRK(PCI_VENDOR_ID_HP, 0x30f7, "HP dv4", STAC_HP_DV5), SND_PCI_QUIRK(PCI_VENDOR_ID_HP, 0x30fc, -- 1.6.1.3

2 1