[alsa-devel] 1.0.15rc3 patch_analog.s bug + fix
Ma Begaj
derliebegott at gmail.com
Tue Oct 16 14:14:47 CEST 2007
Hi,
sorry for partly OT and interrupting your conversation. is there
somewhere a patch which corrects muting for AD1986A when playing 2ch
sound when 6ch is selected in the alsamixer?
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3485
Thanks
M.
2007/10/16, Takashi Iwai <tiwai at suse.de>:
> At Tue, 16 Oct 2007 10:58:19 +0300,
> Vasily Khoruzhick wrote:
> >
> > > > Hmm, the code logic looks correct to me. If HP jack is present, the
> > > > internal speaker should be muted. Otherwise it follows the state of
> > > > HP jack (muted/unmuted, that acts as a master switch).
> >
> > Original patch:
> > present = snd_hda_codec_read(codec, 0x1a, 0,
> > AC_VERB_GET_PIN_SENSE, 0) & 0x80000000;
> >
> > Code in alsa (patch-analog.c line 614):
> >
> > present = snd_hda_codec_read(codec, 0x1a, 0, AC_VERB_GET_PIN_SENSE, 0);
> > spec->jack_present = (present & 0x80000000) != 0;
> >
> > Seems to have quite different logic, don't it?
>
> I don't see your point. It's the same logic:
>
> If present has bit 0x80000000, spec->jack_present = 1,
> Otherwise, spec->jack_present = 0.
>
> The problem is that the jack detection of this codec (or specific to
> the laptop) seems inverse from the standard.
>
> > Should be:
> >
> > present = snd_hda_codec_read(codec, 0x1a, 0, AC_VERB_GET_PIN_SENSE, 0);
> > spec->jack_present = (present & 0x80000000) == 0;
> >
> > P.S. It's very sad that you've released 1.0.15 with that bug :(
>
> Yes, it's sad. Too late recognized. We'll likely have 1.0.15a
> release soon later, though.
>
>
> Takashi
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
--
Glauben heißt nicht wissen wollen, was wahr ist ... Nietzsche
More information about the Alsa-devel
mailing list