Prev: Fixes a typo from "dev" to "ndev"
Next: [PATCH 1/2] tc6387xb: fix resource reclaim
From: Takashi Iwai on 3 Aug 2010 01:50
At Tue, 3 Aug 2010 01:00:54 -0300,
Leandro Lucarella wrote:
>
> 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=dc1eae256cfac03bf17bf3eb016e3a6423d3f9d5
>
> 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).

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?
Please give alsa-info.sh output.


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

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


> 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).

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


Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Takashi Iwai on 3 Aug 2010 09:50
At Tue, 3 Aug 2010 10:44:09 -0300,
Leandro Lucarella wrote:
>
> > Please give alsa-info.sh output.
>
> I didn't know where to find the script,

It's listed in Documentation/sound/alsa/HD-Audio.txt :)

> 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:
> http://www.alsa-project.org/db/?f=6a051da2475c7246f40c493a310fecb0a1a289f5
>
> > > 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.
> >
> > Hm, this should work. At least, you should have beep mixer & co with
> > this.
>
> 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
> commit message. The problem is the same as the one with the ASUS P5-V?

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Takashi Iwai on 3 Aug 2010 11:30
At Tue, 3 Aug 2010 11:48:10 -0300,
Leandro Lucarella wrote:
>
> Takashi Iwai, el 3 de agosto a las 15:48 me escribiste:
> > At Tue, 3 Aug 2010 10:44:09 -0300,
> > Leandro Lucarella wrote:
> > >
> > > > Please give alsa-info.sh output.
> > >
> > > I didn't know where to find the script,
> >
> > It's listed in Documentation/sound/alsa/HD-Audio.txt :)
>
> Woops!
>
> > > 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:
> > > http://www.alsa-project.org/db/?f=6a051da2475c7246f40c493a310fecb0a1a289f5
> > >
> > > > > 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.
> > > >
> > > > Hm, this should work. At least, you should have beep mixer & co with
> > > > this.
> > >
> > > 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
> > > commit message. The problem is the same as the one with the ASUS P5-V?
> >
> > I can add such a trivial patch manually, of course. But I'd need
> > to know which machine you have (vendor/model name, etc).
>
> Is a desktop box.
> CPU Intel(R) Core(TM)2 Quad CPU Q8400 @ 2.66GHz
> Motherboard Intel Corporation DG41RQ

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: Fixes a typo from "dev" to "ndev"
Next: [PATCH 1/2] tc6387xb: fix resource reclaim