linux-kernel - Re: No more Intel HDA soundcard

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]

Date:	Sun, 01 Mar 2009 11:00:35 +0100
From:	Takashi Iwai <tiwai@...e.de>
To:	Alexey Dobriyan <adobriyan@...il.com>
Cc:	linux-kernel@...r.kernel.org, akpm@...l.org, torvalds@...l.org
Subject: Re: No more Intel HDA soundcard

At Sat, 28 Feb 2009 21:44:12 +0300,
Alexey Dobriyan wrote:
> 
> The following commit results in soundcard detection failing with -EBUSY
> 
> commit 34c25350acfc792747e861d1e36874abf7e73255
> Author: Takashi Iwai <tiwai@...e.de>
> Date:   Tue Oct 28 11:38:58 2008 +0100
> 
>     ALSA: hda - Remove old codec-probe limitation
>     
>     Removed the old workaround to avoid the non-existing codec slot.
>     The current code should work without that workaround.  If any,
>     we can add a quirk table.
>     
>     Signed-off-by: Takashi Iwai <tiwai@...e.de>
> 
> I fixed up this (and got sound back) but suspect the following is not right.
> 
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -1224,6 +1224,7 @@ static void azx_stop_chip(struct azx *chip);
>  
>  /* number of codec slots for each chipset: 0 = default slots (i.e. 4) */
>  static unsigned int azx_max_codecs[AZX_NUM_DRIVERS] __devinitdata = {
> +	[AZX_DRIVER_ICH] = 3,
>  	[AZX_DRIVER_TERA] = 1,
>  };

Right, it's no good solution as you expected ;)
The problem is that your patch will disable the all HDMI devices that are
newly supported on 2.6.29.

Is your machine Toshiba X200, found in bug 12735 ??
	http://bugzilla.kernel.org/show_bug.cgi?id=12735

If yes, could you try the patch below?

Also, it'd be helpful if you can try sound git tree (master branch):
    git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
that contains the very latest driver.  This should work as is.


thanks,

Takashi

---
>From ea525d89f74737ebc3674184c6d3c30f9e7c7a6f Mon Sep 17 00:00:00 2001
From: Takashi Iwai <tiwai@...e.de>
Date: Sun, 1 Mar 2009 10:55:44 +0100
Subject: [PATCH] ALSA: hda - Add probe_mask default for Toshiba laptop with ALC268

Some Toshiba laptops have another ALC268 codec on slot#3 that conflicts
with the primary codec.  The codec#3 is for the digital I/O, and should
be fixed by the driver, but it'd need a bunch of changes.

So, let's fix the probe problem temporarily by setting the default
probe_mask value.

Reference: kernel bugzilla #12735
	http://bugzilla.kernel.org/show_bug.cgi?id=12735

Signed-off-by: Takashi Iwai <tiwai@...e.de>
---
 sound/pci/hda/hda_intel.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index c8d9178..5e909e0 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -2095,6 +2095,8 @@ static struct snd_pci_quirk probe_mask_list[] __devinitdata = {
 	SND_PCI_QUIRK(0x1028, 0x20ac, "Dell Studio Desktop", 0x01),
 	/* including bogus ALC268 in slot#2 that conflicts with ALC888 */
 	SND_PCI_QUIRK(0x17c0, 0x4085, "Medion MD96630", 0x01),
+	/* conflict of ALC268 in slot#3 (digital I/O); a temporary fix */
+	SND_PCI_QUIRK(0x1179, 0xff00, "Toshiba laptop", 0x03),
 	{}
 };
 
-- 
1.6.1.3

--
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/