linux-kernel - Re: [patch 27/28] ALSA: hda - Add missing vmaster initialization for ALC269

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

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/