Re: ALC1150 broken on SuperMicro X10SAT

[Takashi Iwai <tiwai@suse.de>]

At Mon, 01 Sep 2014 20:07:26 -0700,
Dana Goyette wrote:
> 
> On 09/01/2014 06:17 AM, Takashi Iwai wrote:
> > What about the patch below?  It might be that only one of two changes
> > is needed.
> >
> >
> > thanks,
> >
> > Takashi
> >
> > ---
> > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> > index d446ac3137b3..1ba22fb527c2 100644
> > --- a/sound/pci/hda/patch_realtek.c
> > +++ b/sound/pci/hda/patch_realtek.c
> > @@ -328,6 +328,7 @@ static void alc_auto_init_amp(struct hda_codec *codec, int type)
> >   		case 0x10ec0885:
> >   		case 0x10ec0887:
> >   		/*case 0x10ec0889:*/ /* this causes an SPDIF problem */
> > +		case 0x10ec0900:
> >   			alc889_coef_init(codec);
> >   			break;
> >   		case 0x10ec0888:
> > @@ -2350,6 +2351,7 @@ static int patch_alc882(struct hda_codec *codec)
> >   	switch (codec->vendor_id) {
> >   	case 0x10ec0882:
> >   	case 0x10ec0885:
> > +	case 0x10ec0900:
> >   		break;
> >   	default:
> >   		/* ALC883 and variants */
> >
> I've applied the patch, and that fixes it without having to specify a 
> model.  Thanks!
> 
> For reference, here's alsa-info after the patch:
> 
> http://www.alsa-project.org/db/?f=d3a135bd31671e545067821bbd2cacc9d8f6fff8

Thanks for a quick test.
FWIW, I applied the patch below to sound git tree now.


Takashi

-- 8< --
From: Takashi Iwai <tiwai@suse.de>
Subject: [PATCH] ALSA: hda - Fix COEF setups for ALC1150 codec

ALC1150 codec seems to need the COEF- and PLL-setups just like its
compatible ALC882 codec.  Some machines (e.g. SunMicro X10SAT) show
the problem like too low output volumes unless the COEF setup is
applied.

Reported-and-tested-by: Dana Goyette <danagoyette@gmail.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
 sound/pci/hda/patch_realtek.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index d446ac3137b3..1ba22fb527c2 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -328,6 +328,7 @@ static void alc_auto_init_amp(struct hda_codec *codec, int type)
 		case 0x10ec0885:
 		case 0x10ec0887:
 		/*case 0x10ec0889:*/ /* this causes an SPDIF problem */
+		case 0x10ec0900:
 			alc889_coef_init(codec);
 			break;
 		case 0x10ec0888:
@@ -2350,6 +2351,7 @@ static int patch_alc882(struct hda_codec *codec)
 	switch (codec->vendor_id) {
 	case 0x10ec0882:
 	case 0x10ec0885:
+	case 0x10ec0900:
 		break;
 	default:
 		/* ALC883 and variants */
-- 
2.1.0