Date:	Wed, 16 Apr 2008 13:01:44 +0200
From:	Takashi Iwai <tiwai@...e.de>
To:	Kasper Sandberg <lkml@...anurb.dk>
Cc:	LKML Mailinglist <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] make alsa use corect patch for Realtek ALC889A

At Sat, 12 Apr 2008 18:17:23 +0200,
Kasper Sandberg wrote:
> 
> Hello..
> I have a gigabyte X48-DQ6 motherboard, specification url:
> http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ClassValue=Motherboard&ProductID=2763&ProductName=GA-X48-DQ6
> they clearly state it uses a realtek alc 889a.
> 
> google around, it seems lots of people which has this chip gets it
> detected as alc885, including me.
> 
> now, audio works (atleast analog out, havent tested the rest), allthough
> i couldnt get the supposed feature of separate channel to front panel to
> work(doesent matter much to me though)
> 
> interrested in fixing this just for the sake of fixing, i ventured into
> patch_realtek.c, and found the structure that defined patches for hda
> audio.
> 
> i inspected my /proc/asound/card0/codec#2 file (only one except the id
> file), and found the following information:
> Codec: Realtek ALC885
> Address: 2
> Vendor Id: 0x10ec0885
> Subsystem Id: 0x1458a002
> Revision Id: 0x100103
> 
> i then modified patch_realtek.c to detect mine as ALC889A, and assign
> the correct patch to it (well, the one used for 889).
> 
> After rebuilding, it now uses patch 883 instead of 882 as it did before.
> i dont appear to have recieved any separate front panel channel,
> however, it still seems more correct now, and is probably wise to keep
> for when 883 is improved to support the hardware fully.
> 
> the only thing i notice has changed, is the message i get from the
> kernel, before i changed, it said:
> hda_codec: Unknown model for ALC882, trying auto-probe from BIOS...
> now it says:
> hda_codec: Unknown model for ALC883, trying auto-probe from BIOS...
> but i guess if it has to spew out such a line, it now spews out the
> correct one.
> 
> analog audio works, so thats fine..
> 
> the patch(i apologize if my mailer messes up whitespace, but its so
> simple that it doesent matter):

The change looks OK (although I'd move the line just before ALC885
entry).  I have no datasheet of ALC889A, but rumor says that it's
almost compatible with ALC888.  So patch_alc883 seems to be a better
choice than now.

I'll fix it on ALSA tree now.

Thanks!


Takashi

> --- /usr/src/linux/sound/pci/hda/patch_realtek.c        2008-04-06
> 02:31:52.840247789 +0200
> +++ /chroot/kernelgentoo/usr/src/linux-2.6.24/sound/pci/hda/patch_realtek.c     2008-04-12 17:09:58.308101686 +0200
> @@ -13759,6 +13759,7 @@
>         { .id = 0x10ec0662, .rev = 0x100101, .name = "ALC662 rev1",
>           .patch = patch_alc662 },
>         { .id = 0x10ec0880, .name = "ALC880", .patch = patch_alc880 },
> +       { .id = 0x10ec0885, .rev = 0x100103, .name = "ALC889A", .patch =
> patch_alc883 },
>         { .id = 0x10ec0882, .name = "ALC882", .patch = patch_alc882 },
>         { .id = 0x10ec0883, .name = "ALC883", .patch = patch_alc883 },
>         { .id = 0x10ec0885, .name = "ALC885", .patch = patch_alc882 },
> 
> 
> 
> i guess some alsa people should be cc'ed, i couldnt find the relevant
> list though, is it alsa-devel@...a-project.org ?
> 
> 
> mvh / regards
> Kasper Sandberg
> 
> --
> 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/
> 
--
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/

Powered by blists - more mailing lists