Oh my god!! It actually works!!! :) At first speaker-test was not working, but with

[kbs1@localhost a]$ speaker-test -c4 -twav -Dsurround40 <--------- THIS

speaker-test 1.0.23

Playback device is surround40 Stream parameters are 48000Hz, S16_LE, 4 channels WAV file(s) Rate set to 48000Hz (requested 48000Hz) Buffer size range from 32 to 8192 Period size range from 16 to 4096 Using max buffer size 8192 Periods = 4 was set period_size = 2048 was set buffer_size = 8192 0 - Front Left 1 - Front Right 3 - Rear Right <-------------- THIS PLAYS 2 - Rear Left <-------------- AND THIS PLAYS!! Time per period = 5.718945 0 - Front Left ^C [kbs1@localhost a]$

Finally! My audio experience is now at a whole new level. Thank you very much, I will post this patch to alsa-bugtracker and close the bug, I was starting to think that this is a kernel problem, but now it just works GREAT.

Maybe you can add this patch to official alsa-driver version on next release.

Have a nice day! Best Regards Viliam Kubis

On Mon, 12 Jul 2010 08:57:59 +0200, Clemens Ladisch clemens@ladisch.de wrote:

Viliam Kubis wrote:

...

I have tried the patch ..., but the surround does not work.

Okay, next try:

--- a/sound/pci/hda/patch_via.c +++ b/sound/pci/hda/patch_via.c @@ -552,24 +552,30 @@ static void via_auto_init_hp_out(struct } } +static int is_smart51_pins(struct via_spec *spec, hda_nid_t pin);

static void via_auto_init_analog_input(struct hda_codec *codec) { struct via_spec *spec = codec->spec;

• unsigned int ctl; int i; for (i = 0; i < AUTO_PIN_LAST; i++) { hda_nid_t nid = spec->autocfg.input_pins[i];

• if (!nid)
  

• 	continue;
  

• if (spec->smart51_enabled && is_smart51_pins(spec, nid))
  

• 	ctl = PIN_OUT;
  

• else if (i <= AUTO_PIN_FRONT_MIC)
  

• 	ctl = PIN_VREF50;
  

• else
  

• 	ctl = PIN_IN;
  

  snd_hda_codec_write(codec, nid, 0,

• 		    AC_VERB_SET_PIN_WIDGET_CONTROL,
  

• 		    (i <= AUTO_PIN_FRONT_MIC ?
  

• 		     PIN_VREF50 : PIN_IN));
  

• 		    AC_VERB_SET_PIN_WIDGET_CONTROL, ctl);
  

  }

} -static int is_smart51_pins(struct via_spec *spec, hda_nid_t pin);

static void set_pin_power_state(struct hda_codec *codec, hda_nid_t nid, unsigned int *affected_parm) { @@ -658,6 +664,8 @@ static void set_jack_power_state(struct /* PW0 (19h), SW1 (18h), AOW1 (11h) */ parm = AC_PWRST_D3; set_pin_power_state(codec, 0x19, &parm);

• if (spec->smart51_enabled)
  

• 	parm = AC_PWRST_D0;
  

  snd_hda_codec_write(codec, 0x18, 0, AC_VERB_SET_POWER_STATE, parm); snd_hda_codec_write(codec, 0x11, 0, AC_VERB_SET_POWER_STATE,

@@ -667,6 +675,8 @@ static void set_jack_power_state(struct if (is_8ch) { parm = AC_PWRST_D3; set_pin_power_state(codec, 0x22, &parm);

• 	if (spec->smart51_enabled)
  

• 		parm = AC_PWRST_D0;
  snd_hda_codec_write(codec, 0x26, 0,
  		    AC_VERB_SET_POWER_STATE, parm);
  snd_hda_codec_write(codec, 0x24, 0,
  

@@ -3915,6 +3925,13 @@ static int vt1708S_auto_fill_dac_nids(st } }

• /* for Smart 5.1, line/mic inputs double as output pins */
• if (cfg->line_outs == 1) {

• spec->multiout.num_dacs = 3;
  

• spec->multiout.dac_nids[AUTO_SEQ_SURROUND] = 0x11;
  

• spec->multiout.dac_nids[AUTO_SEQ_CENLFE] = 0x24;
  

• }
• return 0;

} @@ -3932,7 +3949,8 @@ static int vt1708S_auto_create_multi_out for (i = 0; i <= AUTO_SEQ_SIDE; i++) { nid = cfg->line_out_pins[i];

• if (!nid)
  

• /* for Smart 5.1, there are always at least six channels */
  

• if (!nid && i > AUTO_SEQ_CENLFE)
  continue;
  

  nid_vol = nid_vols[i];