[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20090814171453.GA21806@suse.de>
Date: Fri, 14 Aug 2009 10:14:53 -0700
From: Greg KH <gregkh@...e.de>
To: Takashi Iwai <tiwai@...e.de>
Cc: Linus Torvalds <torvalds@...ux-foundation.org>,
linux-kernel@...r.kernel.org, stable@...nel.org,
stable-review@...nel.org, akpm@...ux-foundation.org,
alan@...rguk.ukuu.org.uk
Subject: Re: [patch 27/28] ALSA: hda - Add missing vmaster initialization
for ALC269
On Fri, Aug 14, 2009 at 08:56:28AM +0200, Takashi Iwai wrote:
> At Thu, 13 Aug 2009 13:12:22 -0700,
> Greg KH wrote:
> >
> > On Thu, Aug 13, 2009 at 12:59:05PM -0700, Linus Torvalds wrote:
> > >
> > >
> > > On Thu, 13 Aug 2009, Greg KH wrote:
> > > >
> > > > From: Takashi Iwai <tiwai@...e.de>
> > > >
> > > > commit 100d5eb36ba20dc0b99a17ea2b9800c567bfc3d1 upstream.
> > > >
> > > > Without the initialization of vmaster NID, the dB information got
> > > > confused for ALC269 codec.
> > > >
> > > > Reference: Novell bnc#527361
> > > > https://bugzilla.novell.com/show_bug.cgi?id=527361
> > > >
> > > > Signed-off-by: Takashi Iwai <tiwai@...e.de>
> > > > Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
> > > >
> > > > ---
> > > > sound/pci/hda/patch_realtek.c | 2 ++
> > > > 1 file changed, 2 insertions(+)
> > > >
> > > > --- a/sound/pci/hda/patch_realtek.c
> > > > +++ b/sound/pci/hda/patch_realtek.c
> > > > @@ -13767,6 +13767,8 @@ static int patch_alc861vd(struct hda_cod
> > > >
> > > > spec->vmaster_nid = 0x02;
> > > >
> > > > + spec->vmaster_nid = 0x02;
> > > > +
> > > > codec->patch_ops = alc_patch_ops;
> > >
> > > You seem to have already applied this one earlier ;)
> >
> > Heh, good catch, I've now removed this one from the queue.
>
> Well, the patch has to be applied to another place, namely in the
> function patch_alc269(), but patch seems confused because the offset
> is too big.
>
> As I'm off today, I'm going to check it and resend the patch for
> 2.6.27 tomorrow.
Thanks, that would be great.
greg k-h
--
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