[alsa-devel] Incorrect model selection for HDA Intel based card.

[Travis Place]

On Tue, 20 May 2008 1:54:42 am Takashi Iwai wrote:
> At Tue, 20 May 2008 01:43:31 +1000,
>
> Travis Place wrote:
> > On Mon, 19 May 2008 10:09:00 pm Takashi Iwai wrote:
> > > At Mon, 19 May 2008 04:19:14 +1000,
> > >
> > > Travis Place wrote:
> > > > Just had a user in #alsa on freenode, with an Asus P5GD1 mainboard.
> > > > They couldnt seem to get surround sound working at all.
> > > > It has an ALC880 based HDA-Intel sound device on board.
> > > >
> > > > Now, the problem lies in the Susbsystem ID's (1043:814e).
> > > > These match in patch_realtek.c as follows:
> > > >
> > > > SND_PCI_QUIRK(0x1043, 0x814e, "ASUS", ALC880_ASUS)
> > > >
> > > > The users mainboard (The ASUS P5GD1) actually has 6 output jacks,
> > > > aswell as SPDIF, not the standard 3-jacks that the "ASUS" model quirk
> > > > implies.
> > > >
> > > > I talked this user through forcing the driver to use '6stack-digout'
> > > > for his sound device, and things all work fine now.
> > >
> > > Actually, this was changed originally from ALC880_5STACK for the
> > > exactly same mobo (changeset 4798).  I guess 6stack-digout wasn't
> > > tested by that user.
> > >
> > > So, if 6stack-digout is confirmed to match better than now, it's
> > > fairly safe to replace the line.  Care to send a patch with a proper
> > > changelog?
> > >
> > >
> > > thanks,
> > >
> > > Takashi
> > > _______________________________________________
> > > Alsa-devel mailing list
> > > Alsa-devel at alsa-project.org
> > > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> >
> > As previously mentioned, it appears the HG code has an entry for the
> > "ASUS P5GD1" mainboard. But its onboard sound vendor/device Ids dont
> > match those of the person i was helping. I have double checked the
> > pastebin post, and confirmed the mainboard model with the user though.
> >
> > The _original_ alsa-info.sh run (before the model was forced to
> > 6stack-digout) is as follows..
> >
> > http://pastebin.ca/1021995
> >
> > There it shows the vendor/device ids as 0x1043, 0x814e .
>
> 1043:814e is also P5GD1.
> 	http://www.linuxquestions.org/questions/showthread.php?p=2556497#post25564
>97
>
> It was changed to ALC880_ASUS based on the report in the above.
>
> > So according to the current HG code, those IDs would select the "asus"
> > model. Also, in the same, current HG code, we have..
> >
> > SND_PCI_QUIRK(0x1043, 0x8196, "ASUS P5GD1", ALC880_6ST)
> >
> > This shows the correct model name for the mainboard, but (from my
> > experience) wrong vendor/device ids.
> >
> > Perhaps we have to check with the person who originally added that P5GD1
> > quirk ?
> >
> > I have made up a patch, but want to confirm all this information before
> > proceeding. Please let me know your thoughts.
>
> I believe it's fine to change to 6stack model if the tester's board is
> also P5GD1 indeed.  Of course, it's possible that ASUS may change the
> hardware design completely for the same model number with PCI SSID,
> but I hope they aren't so crazy :)
>
>
> thanks,
>
> Takashi
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Below (as well as attached) is my patch, for the 0x814e variant.


diff -r e2b60934d1c8 pci/hda/patch_realtek.c
--- a/pci/hda/patch_realtek.c	Mon May 19 12:32:25 2008 +0200
+++ b/pci/hda/patch_realtek.c	Tue May 20 01:50:04 2008 +1000
@@ -2981,7 +2981,7 @@ static struct snd_pci_quirk alc880_cfg_t
 	/* SND_PCI_QUIRK(0x1043, 0x1964, "ASUS", ALC880_ASUS_DIG), */
 	SND_PCI_QUIRK(0x1043, 0x1973, "ASUS", ALC880_ASUS_DIG),
 	SND_PCI_QUIRK(0x1043, 0x19b3, "ASUS", ALC880_ASUS_DIG),
-	SND_PCI_QUIRK(0x1043, 0x814e, "ASUS", ALC880_ASUS),
+	SND_PCI_QUIRK(0x1043, 0x814e, "ASUS P5GD1 w/SPDIF", ALC880_6ST_DIG),
 	SND_PCI_QUIRK(0x1043, 0x8181, "ASUS P4GPL", ALC880_ASUS_DIG),
 	SND_PCI_QUIRK(0x1043, 0x8196, "ASUS P5GD1", ALC880_6ST),
 	SND_PCI_QUIRK(0x1043, 0x81b4, "ASUS", ALC880_6ST),

Changelog should be something like:

Corrected the model assignment for the ASUS P5GD1 w/SPDIF after reports of 
surround sound not being possible.

Let me know if you need a more detailed changelog entry.

Travis Place

-- 
Happiness is like peeing your pants. Everyone can see it, but only you can 
feel its warmth 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: P5GD1.patch
Type: text/x-diff
Size: 711 bytes
Desc: not available
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20080520/aaaa06a3/attachment.bin