Re: [PATCH] [RESEND] FIX to not expose SPDIF on Thinkpad X301, since it does not have the ability to use SPDIF

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sorry about that .. not sure why 2 times it screwed up. I'll get it
right on my next send. The last one was missing parts of the
diff..again.

			Jerone

On Tue, 2010-08-03 at 08:21 +0200, Takashi Iwai wrote:
> At Tue, 03 Aug 2010 08:16:59 +0200,
> David Henningsson wrote:
> > 
> > I think Takashi wants you to use "git format-patch" to generate the
> > patch. At least I do so and that has been working well so far.
> 
> Right.  The previous generated patches are really broken.  You can't
> apply it via patch program, even manually.  If you are using GIT,
> better to let GIT working such a boring job :)
> 
> 
> Takashi
> 
> > // David
> > 
> > 2010-08-03 08:05, Jerone Young skrev:
> > > The Lenovo X301 does not have the ability to connect to a docking
> > > station to use the SPDIF port. It also does not have the ability to do
> > > SPDIF though the headphone jack or Display Port jacks.
> > > 
> > > This patch fixes it so this is not exposed for the X301 and users do
> > > think it has the ability to do SPDIF.
> > > 
> > > I tested both headphone & display port jacks and it is not there. I have
> > > tested this patch and it works great.
> > > 
> > > Also to add the other Thinkpads have different subsystem codec IDs. 
> > > Here are examples:
> > > 
> > > X301:
> > > http://launchpadlibrarian.net/31561902/Card0.Codecs.codec.0.txt
> > > 
> > > X200:
> > > http://launchpadlibrarian.net/49055036/Card0.Codecs.codec.0.txt
> > > 
> > > W500:
> > > http://launchpadlibrarian.net/36276057/Card0.Codecs.codec.0.txt
> > > 
> > > 
> > > Signed-off-by: Jerone Young <jerone.young@xxxxxxxxxxxxx>
> > > 
> > > diff --git a/pci/hda/patch_conexant.c b/pci/hda/patch_conexant.c
> > > index c49030b..abc320a 100644
> > > @@ -2004,6 +2006,11 @@ static int patch_cxt5051(struct hda_codec *codec)
> > >  		break;
> > >  	case CXT5051_LENOVO_X200:
> > >  		spec->init_verbs[0] = cxt5051_lenovo_x200_init_verbs;
> > > +		/* Thinkpad X301 does not have S/PDIF wired and no ability
> > > +		   to use a docking station. */
> > > +		if (codec->subsystem_id == 0x17aa211f) {
> > > +			spec->multiout.dig_out_nid = 0; 
> > > +		}
> > >  		break;
> > >  	case CXT5051_F700:
> > >  		spec->init_verbs[0] = cxt5051_f700_init_verbs;
> > > 
> > > 
> > > _______________________________________________
> > > Alsa-devel mailing list
> > > Alsa-devel@xxxxxxxxxxxxxxxx
> > > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> > > 
> > 
> > 
> > -- 
> > David Henningsson, Canonical Ltd.
> > http://launchpad.net/~diwic
> > 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux