From: David Henningsson <david.henningsson@canonical.com>
To: Thanos Kyritsis <djart@linux.gr>, Takashi Iwai <tiwai@suse.de>,
"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>
Subject: Re: Possible regression in patch_realtek.c
Date: Mon, 12 Jan 2015 07:33:14 +0100 [thread overview]
Message-ID: <54B36AAA.3020905@canonical.com> (raw)
In-Reply-To: <CAKovSWXeXsCoCVOwe_srtKYLW6yPJfc_MihTSRGNT4iWS=e1yw@mail.gmail.com>
On 2015-01-11 22:11, Thanos Kyritsis wrote:
> 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?
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.
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
next parent reply other threads:[~2015-01-12 6:33 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CAKovSWXeXsCoCVOwe_srtKYLW6yPJfc_MihTSRGNT4iWS=e1yw@mail.gmail.com>
2015-01-12 6:33 ` David Henningsson [this message]
2015-01-12 6:45 ` Possible regression in patch_realtek.c Raymond Yau
2015-01-12 19:17 ` Thanos Kyritsis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54B36AAA.3020905@canonical.com \
--to=david.henningsson@canonical.com \
--cc=alsa-devel@alsa-project.org \
--cc=djart@linux.gr \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.