Re: [PATCH] make alsa use corect patch for Realtek ALC889A

From: Takashi Iwai
Date: Wed Apr 16 2008 - 07:01:55 EST


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@xxxxxxxxxxxxxxxx ?
>
>
> mvh / regards
> Kasper Sandberg
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/