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

[David Henningsson <david.henningsson@canonical.com>]

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.

// 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@canonical.com>
> 
> 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@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 


-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic