[alsa-devel] Possible regression in patch_realtek.c

[Raymond Yau]

>>
>> Hello,
>>
>> I hope it's ok to e-mail you directly instead of going through an
>> official bug tracker, sorry in advance otherwise.
>
>
> Ok. I have added alsa-devel to CC.
>
>> I'd like your opinion on a patch of yours that dates a year back:
>>
>>
https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/sound/pci/hda/patch_realtek.c?id=b2c53e206967d01fd4fb6dd525f89ae738beb2e6
>> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1248116
>>
>> This patch has a negative impact for my hardware. I've completely lost
>> the kernel's front panel jack auto-mute functionality.
>>
>> In detail: my motherboard is Gigabyte Z68AP-D3, utilizing ALC889
>> chipset (http://www.gigabyte.com/products/product-page.aspx?pid=3897)
>> with PCI id: 0x1458, 0xa002.
>>
>> Without this patch (that is kernel up to 3.13.x), all is OK, when I
>> plug headphones via the front panel jack, the kernel auto-mutes the
>> speaker-phone jack.
>> With the patch (that is kernel 3.14.x and all later versions),
>> auto-mute does not work. Plugging headphones results on sound getting
>> through both jacks simultaneously.
>> I can, of course, control the volume/mute of these two jacks
>> independently via CLI alsamixer, but I think this is a regression. Do
>> you agree ? I'm noticing that the patch's purpose was to fix a
>> slightly different problem, though.
>> Is it possible to satisfy both requirements for both motherboards ?
>>
>> Can I do something to further help ? If you'd like me to comment on
>> bug #1248116, that will be no problem of course.
>>
>> Thank you in advance for your help and for your contribution towards a
>> better kernel and distros :-)
>
>
> Okay. So the real question is if there's a way we can tell the working
and the faulty ones apart. Until we have found such a way, I'll be hesitant
to revert the patch.
>
> Here's alsa-info for a faulty one:
http://www.alsa-project.org/db/?f=435c404946687b445f6c60da5edc5c3f083bc1da
>
> Both Codec SSID and PCI SSID are 1458:a002 on that machine, codec vendor
ID is 10ec:0899 - maybe we should only apply this quirk for certain codec
vendor IDs, then?

This is bacause all gigabyte motherboards have same pci ssid 1458:a002

All gigabytes with different realtek codecs are affected by several patches

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/log/sound/pci/hda/patch_realtek.c?qt=grep&q=gigabyte

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/sound/pci/hda/patch_realtek.c?id=b2c53e206967d01fd4fb6dd525f89ae738beb2e6

+ SND_PCI_QUIRK(0x1458, 0xa002, "Gigabyte EP45-DS3/Z87X-UD3H",
ALC889_FIXUP_FRONT_HP_NO_PRESENCE),

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/sound/pci/hda/patch_realtek.c?id=78f8baf138311be3e170526388b0530a172bdbff

+ SND_PCI_QUIRK(0x1458, 0xa002, "GA-MA790X", 1),

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/sound/pci/hda/patch_realtek.c?id=8f2392142346f2754c8292a94cc62a157ed1e093

+ SND_PCI_QUIRK(0x1458, 0xa002, "Gigabyte EP45-DS3", ALC889_FIXUP_CD),

>
> Anyhow, you can bring back jack detection for yourself (as a workaround)
by reconfiguring the codec. This is easiest done through hda-jack-retask
(available in the alsa-tools repository). In hda-jack-retask click
"advanced override", then make sure pin 0x1b has jack detect available.
>