Re: Regression 2.6.35: Intel N10/ICH 7 Family HDA (Realtek ALC662 rev1) beep broken

Previous thread: [PATCH 1/2]drivers/usb/core/sysfs.c Fix variable 'retval' set but not used by Justin P. Mattock on Monday, August 2, 2010 - 9:26 pm. (4 messages)

Next thread: [PATCH 2.6.32/stable] CIFS: Remove __exit mark from cifs_exit_dns_resolver() by Michael Neuling on Monday, August 2, 2010 - 10:03 pm. (4 messages)
From: Leandro Lucarella
Date: Monday, August 2, 2010 - 9:00 pm

Hi. I've upgraded from kernel 2.6.34 to 2.6.35 and the beep through the
sound card stopped working. I found a very similar problem[1]
(commit[2]).

[1] http://lkml.org/lkml/2010/7/28/228
[2] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=dc1eae...

I've tried to manually enabling the beep with the steps in the e-mail
thread I found:

# echo -n 1 > /sys/class/sound/hwC0D2/subsystem_id
# echo -n 1 > /sys/class/sound/hwC0D2/reconfig

But it didn't work (the beep still goes through the pcspkr).

I've tried to naively (I'm not a kernel hacker, so I was just guessing) add a
new entry with the quirks beep_white_list by copying the entry for ASUS like
this:

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 596ea2f..ca3ed12 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -5183,6 +5183,7 @@ static void fillup_priv_adc_nids(struct hda_codec *codec, hda_nid_t *nid
 
 static struct snd_pci_quirk beep_white_list[] = {
        SND_PCI_QUIRK(0x1043, 0x829f, "ASUS", 1),
+       SND_PCI_QUIRK(0x8086, 0xd613, "INTEL", 1),
        {}
 };
 
But it didn't work either. Then I noticed, if I'm understanding the source code
(sound/pci/hda/patch_realtek.c) and documentation
(Documentation/sound/alsa/HD-Audio-Models.txt) right, that my card is not
present in the models listing, but it worked fine in 2.6.34 (and previous)
kernel(s).

I would appreciate if somebody can give me some hints on how to get the beep
back.

Thank you.


Here is some data about my card, please let me know (Cc me please) if you need
any other information.

# lspci -vvs00:1b.0
00:1b.0 Audio device: Intel Corporation N10/ICH 7 Family High Definition Audio Controller (rev 01)
	Subsystem: Intel Corporation Device d613
	Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx+
	Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- ...
From: Takashi Iwai
Date: Monday, August 2, 2010 - 10:42 pm

At Tue, 3 Aug 2010 01:00:54 -0300,

So, you want to hook the beep over codec but it doesn't work, right.
Do you get "Beep" mixer elements and the dedicated input device?

Hm, this should work.  At least, you should have beep mixer & co with

This is no problem.  The model entry is only for special cases.
For normal machines, no quirks are needed.


Takashi
--

From: Leandro Lucarella
Date: Tuesday, August 3, 2010 - 6:44 am

No, I don't think I have any of them (cat cat /proc/bus/input/devices at the

I didn't know where to find the script, Google gave me this:
http://git.alsa-project.org/?p=alsa-driver.git;a=blob_plain;f=utils/alsa-info.sh
So this is what I've use. I've shared the results at:

Well, my bad, I booted the wrong kernel when trying the patch. The patch
works correctly, so I'll be very glad if it hits Linus kernel soon :)

I could send you the patch but I don't know exactly what to put in the

OK, thanks for the help.

I hope the patch is committed into the main kernel soon.



# cat /proc/bus/input/devices
I: Bus=0011 Vendor=0001 Product=0001 Version=ab41
N: Name="AT Translated Set 2 keyboard"
P: Phys=isa0060/serio0/input0
S: Sysfs=/devices/platform/i8042/serio0/input/input0
U: Uniq=
H: Handlers=sysrq kbd event0 
B: EV=120013
B: KEY=402000000 3803078f800d001 feffffdfffefffff fffffffffffffffe
B: MSC=10
B: LED=7

I: Bus=0010 Vendor=001f Product=0001 Version=0100
N: Name="PC Speaker"
P: Phys=isa0061/input0
S: Sysfs=/devices/platform/pcspkr/input/input1
U: Uniq=
H: Handlers=kbd event1 
B: EV=40001
B: SND=6

I: Bus=0019 Vendor=0000 Product=0001 Version=0000
N: Name="Power Button"
P: Phys=PNP0C0C/button/input0
S: Sysfs=/devices/LNXSYSTM:00/LNXSYBUS:00/PNP0C0C:00/input/input2
U: Uniq=
H: Handlers=kbd event2 
B: EV=3
B: KEY=10000000000000 0

I: Bus=0019 Vendor=0000 Product=0001 Version=0000
N: Name="Power Button"
P: Phys=LNXPWRBN/button/input0
S: Sysfs=/devices/LNXSYSTM:00/LNXPWRBN:00/input/input3
U: Uniq=
H: Handlers=kbd event3 
B: EV=3
B: KEY=10000000000000 0

I: Bus=0003 Vendor=09da Product=022b Version=0110
N: Name="A4Tech Wireless Battery Free Optical Mouse"
P: Phys=usb-0000:00:1d.2-1/input0
S: Sysfs=/devices/pci0000:00/0000:00:1d.2/usb4/4-1/4-1:1.0/input/input4
U: Uniq=
H: Handlers=mouse0 event4 
B: EV=17
B: KEY=ff0000 0 0 0 0
B: REL=303
B: MSC=10

I: Bus=0019 Vendor=0000 Product=0006 Version=0000
N: Name="Video Bus"
P: ...
From: Takashi Iwai
Date: Tuesday, August 3, 2010 - 6:48 am

At Tue, 3 Aug 2010 10:44:09 -0300,


I can add such a trivial patch manually, of course.  But I'd need
to know which machine you have (vendor/model name, etc).


thanks,

Takashi
--

From: Leandro Lucarella
Date: Tuesday, August 3, 2010 - 7:48 am

Is a desktop box.
CPU Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz
Motherboard Intel Corporation DG41RQ

Let me know if you need any other information.


Thank you.

Here is the output of lshw:

# lshw -short
H/W path               Device       Class       Description
===========================================================
                                    system      Desktop Computer
/0                                  bus         DG41RQ
/0/0                                memory      64KiB BIOS
/0/4                                processor   Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz
/0/4/5                              memory      32KiB L1 cache
/0/4/6                              memory      2MiB L2 cache
/0/2a                               memory      2GiB System Memory
/0/2a/0                             memory      DIMM Synchronous [empty]
/0/2a/1                             memory      2GiB DIMM DDR2 Synchronous 800 MHz (1.2 ns)
/0/100                              bridge      4 Series Chipset DRAM Controller
/0/100/1                            bridge      4 Series Chipset PCI Express Root Port
/0/100/2                            display     4 Series Chipset Integrated Graphics Controller
/0/100/1b                           multimedia  N10/ICH 7 Family High Definition Audio Controller
/0/100/1c                           bridge      N10/ICH 7 Family PCI Express Port 1
/0/100/1c.1                         bridge      N10/ICH 7 Family PCI Express Port 2
/0/100/1c.1/0          eth0         network     RTL8111/8168B PCI Express Gigabit Ethernet controller
/0/100/1d                           bus         N10/ICH 7 Family USB UHCI Controller #1
/0/100/1d.1                         bus         N10/ICH 7 Family USB UHCI Controller #2
/0/100/1d.2                         bus         N10/ICH 7 Family USB UHCI Controller #3
/0/100/1d.3                         bus         N10/ICH 7 Family USB UHCI Controller #4
/0/100/1d.7                         bus         N10/ICH 7 ...
From: Takashi Iwai
Date: Tuesday, August 3, 2010 - 8:24 am

At Tue, 3 Aug 2010 11:48:10 -0300,

OK, it's fine, I just wanted to be sure that it's no other brand.
The fix is now applied to sound tree, and will be included in the next
pull request, then will reach to stable tree.


thanks,

Takashi
--

From: Leandro Lucarella
Date: Tuesday, August 3, 2010 - 9:00 am

Thank you.

-- 
Leandro Lucarella (AKA luca)                     http://llucax.com.ar/
----------------------------------------------------------------------
GPG Key: 5F5A8D05 (F8CD F9A7 BF00 5431 4145  104C 949E BFB6 5F5A 8D05)
----------------------------------------------------------------------
La esperanza es una amiga que nos presta la ilusión.
--

Previous thread: [PATCH 1/2]drivers/usb/core/sysfs.c Fix variable 'retval' set but not used by Justin P. Mattock on Monday, August 2, 2010 - 9:26 pm. (4 messages)

Next thread: [PATCH 2.6.32/stable] CIFS: Remove __exit mark from cifs_exit_dns_resolver() by Michael Neuling on Monday, August 2, 2010 - 10:03 pm. (4 messages)