[PATCH] snd_hda_intel for F/S T4210

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <linux-kernel@...>
Date: Wednesday, August 29, 2007 - 11:34 am

Hi folks,

the patch below, to be applied to sound/pci/hda/patch_sigmatel.c fixes the audio
output on the Fujiutsu/Siemens lifebook T4210 (and probably on others). It is
suitable for the kernel 2.6.23-rc4 (and probably others).

Without the patch, audio fails and the hda driver fails to load with

No available DAC for pin 0x0

However, the indicated pin has no connections in first place and thus should be ignored.
With the patch applied audio output works fine.

Here's the patch:

/* snip */
--- patch_sigmatel_old.c        2007-08-29 17:17:51.000000000 +0200
+++ patch_sigmatel.c    2007-08-29 17:21:25.000000000 +0200
@@ -1250,6 +1250,16 @@
                nid = cfg->line_out_pins[i];
                conn_len = snd_hda_get_connections(codec, nid, conn,
                                                   HDA_MAX_CONNECTIONS);
+               /* FIX THOR: For some hardware (notably the Fujiutsu/Siemens T4210
+               ** lifebook), we have nids with no pins whatsoever. Ignore
+               ** them.
+               */
+               if (conn_len == 0) {
+                 snd_printk(KERN_ERR "%s: Found zero connections for pin 0x%0x "
+                            "ignoring it.\n",__func__,nid);
+                 continue;
+               }
+
                for (j = 0; j < conn_len; j++) {
                        wcaps = snd_hda_param_read(codec, conn[j],
                                                   AC_PAR_AUDIO_WIDGET_CAP);
/* snip */

Greetings,
	Thomas
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] snd_hda_intel for F/S T4210, Thomas Richter, (Wed Aug 29, 11:34 am)
Re: [PATCH] snd_hda_intel for F/S T4210, Takashi Iwai, (Thu Aug 30, 11:04 am)
Re: [PATCH] snd_hda_intel for F/S T4210, Takashi Iwai, (Fri Aug 31, 7:08 am)