Takashi,
I applied your patch (to alsa-driver-hg-20070421) and modprobe'd
snd_hda_intel without probe_mask, but with model=auto. It worked and
detected the ALC861 codec straight away, just like it did with
probe_mask=8. I max'ed all playback and capture volumes , ran arecord
| aplay ,which worked fine. The laptop I have (toshiba A100) only has
builtin speakers and a headphone jack for output (and a mic jack for
input). There are no other jacks. I tried both the speakers and
headphones, both worked (the speakers are muted when the headphone is
connected).
Next I modprobe'd again snd_hda_intel without any parameters this
time, and it worked fine too. the codec was detected and both playback
and capture worked fine.
On 4/24/07, Takashi Iwai <tiwai@xxxxxxx> wrote:
> At Tue, 24 Apr 2007 00:12:46 +1000,
> Ben DiDonc wrote:
> >
> > Here are the two codec files:
> > codec#3-working-14rc1 was obtained after loading snd-hda-intel with
> > probe_mask=8 and model=auto
> > the other was obtained after loading the module without special parameters
>
> Thanks. Let's cut out bugs step by step.
>
> The probing problem with the latest driver might be fixed by the patch
> below. Could you try it and load without probe_mask option?
>
> Also, what output did you test actually? Is the output from the
> built-in speaker (if any), the output from HP-jack or both?
>
>
> Takashi
>
> diff -r aed6a15c9398 pci/hda/hda_intel.c
> --- a/pci/hda/hda_intel.c Mon Apr 23 11:54:41 2007 +0200
> +++ b/pci/hda/hda_intel.c Mon Apr 23 16:18:35 2007 +0200
> @@ -979,7 +979,7 @@ static int __devinit azx_codec_create(st
> static int __devinit azx_codec_create(struct azx *chip, const char *model)
> {
> struct hda_bus_template bus_temp;
> - int c, codecs, err;
> + int c, codecs, audio_codecs, err;
>
> memset(&bus_temp, 0, sizeof(bus_temp));
> bus_temp.private_data = chip;
> @@ -991,16 +991,19 @@ static int __devinit azx_codec_create(st
> if ((err = snd_hda_bus_new(chip->card, &bus_temp, &chip->bus)) < 0)
> return err;
>
> - codecs = 0;
> + codecs = audio_codecs = 0;
> for (c = 0; c < AZX_MAX_CODECS; c++) {
> if ((chip->codec_mask & (1 << c)) & probe_mask) {
> - err = snd_hda_codec_new(chip->bus, c, NULL);
> + struct hda_codec *codec;
> + err = snd_hda_codec_new(chip->bus, c, &codec);
> if (err < 0)
> continue;
> codecs++;
> + if (codec->afg)
> + audio_codecs++;
> }
> }
> - if (!codecs) {
> + if (!audio_codecs) {
> /* probe additional slots if no codec is found */
> for (; c < azx_max_codecs[chip->driver_type]; c++) {
> if ((chip->codec_mask & (1 << c)) & probe_mask) {
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
