linux-kernel - Re: [BUG]3.14-rc2 [PATCH] Revert "ALSA: hda/realtek

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <s5h4n445mm2.wl%tiwai@suse.de>
Date:	Wed, 12 Feb 2014 17:20:21 +0100
From:	Takashi Iwai <tiwai@...e.de>
To:	Martin Kepplinger <martink@...teo.de>
Cc:	perex@...ex.cz, david.henningsson@...onical.com,
	hui.wang@...onical.com, alsa-devel@...a-project.org,
	linux-kernel@...r.kernel.org
Subject: Re: [BUG]3.14-rc2 [PATCH] Revert "ALSA: hda/realtek - Avoid invalid COEFs for ALC271X"

At Wed, 12 Feb 2014 17:09:23 +0100,
Martin Kepplinger wrote:
> 
> This reverts commit d3c56568f43807135f2c2a09582a69f809f0d8b7.
> 
> The reverted commit breaks audio through headphone line out on
> the Acer TravelMate B113 (Type1Sku0) Notebook, my main work
> machine. I don't know much about it but this fixes my problem.
> Bisected and tested.
> 
> Tested-by: Martin Kepplinger <martink@...teo.de>
> Signed-off-by: Martin Kepplinger <martink@...teo.de>

Too bad, we need COEF for some machine but it breaks for some.
Since reverting breaks obviously another machine, we need a different
approach, e.g. checking the machine ID.  Please give alsa-info.sh
output of your machine.


thanks,

Takashi

> ---
>  sound/pci/hda/patch_realtek.c |    3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index d9693ca..0f5af34 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -4434,9 +4434,6 @@ static void alc269_fill_coef(struct hda_codec *codec)
>  
>  	if (spec->codec_variant != ALC269_TYPE_ALC269VB)
>  		return;
> -	/* ALC271X doesn't seem to support these COEFs (bko#52181) */
> -	if (!strcmp(codec->chip_name, "ALC271X"))
> -		return;
>  
>  	if ((alc_get_coef0(codec) & 0x00ff) < 0x015) {
>  		alc_write_coef_idx(codec, 0xf, 0x960b);
> -- 
> 1.7.10.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/