alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: "Viliam Kubis" <viliam.kubis@gmail.com>
To: clemens@ladisch.de,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>
Subject: Re: Smart 5.1 function not working on VIA VT1708S with *three* jacks on mainboard
Date: Fri, 09 Jul 2010 13:04:32 +0200	[thread overview]
Message-ID: <op.vfko1ulyy9cctd@localhost> (raw)
In-Reply-To: <op.vfi12awty9cctd@localhost>

Hello, I have patched the vt1708s_auto_fill_dac_nids function instead. I
have left values 0x11 and 0x12 for surround and cen/lfe, althrough it
might be 0x11 and 0x24 for vt1708S as originally specified in
vt1708s_auto_fill_dac_nids switch statement. The whole function looks
like this:

  static int vt1708S_auto_fill_dac_nids(struct via_spec *spec,
  				     const struct auto_pin_cfg *cfg)
  {
  	int i;
  	hda_nid_t nid;

  	spec->multiout.num_dacs = cfg->line_outs;

  	spec->multiout.dac_nids = spec->private_dac_nids;

  	for (i = 0; i < 4; i++) {
  		nid = cfg->line_out_pins[i];
  		if (nid) {
  			/* config dac list */
  			switch (i) {
  			case AUTO_SEQ_FRONT:
  				spec->multiout.dac_nids[i] = 0x10;
  				break;
  			case AUTO_SEQ_CENLFE:
  				spec->multiout.dac_nids[i] = 0x24;
  				break;
  			case AUTO_SEQ_SURROUND:
  				printk(KERN_INFO "***PATCH CREATING SURROUND DAC_NID");
  				spec->multiout.dac_nids[i] = 0x11;
  				break;
  			case AUTO_SEQ_SIDE:
  				spec->multiout.dac_nids[i] = 0x25;
  				break;
  			}
  		}
  	}
  	
  	if (cfg->line_outs == 1)
  	{
  		printk(KERN_INFO "***PATCH 08/07/10");
  		spec->multiout.num_dacs = 3;
  		spec->multiout.dac_nids[AUTO_SEQ_SURROUND] = 0x11;
  		spec->multiout.dac_nids[AUTO_SEQ_CENLFE] = 0x12;
	}

  	return 0;
  }

  The patch is in effect since the following is visible in dmesg after the
  driver is loaded:

  ***PATCH 08/07/10
  ***PATCH CREATING Surround CONTROL
  ***PATCH HDA max_channels = 6
  ***PATCH playback stream channels_max = 6
  Smart 5.1 Patch installed NEW
  ***PATCH Setting 64 mode [out=64] for pin index 0 nid 26
  ***PATCH Setting 64 mode [out=64] for pin index 1 nid 30
  ***PATCH Setting 64 mode [out=64] for pin index 2 nid 27

  printk(KERN_INFO "***PATCH CREATING SURROUND DAC_NID"); This line does
  not print since cfg->line_out_pins[AUTO_SEQ_SURROUND] is not filled at
  that time, but we fix it in the new if block.

  Surround still does not work (still the same, the rear speakers are
  soundless).

  On Thu, 08 Jul 2010 08:33:34 +0200, Clemens Ladisch <clemens@ladisch.de>
  wrote:

>> _auto_fill_dac_nids
>


-- 
Using Opera's revolutionary e-mail client: http://www.opera.com/mail/

  parent reply	other threads:[~2010-07-09 11:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-30 22:02 Smart 5.1 function not working on VIA VT1708S with *three* jacks on mainboard Viliam Kubis
2010-07-06 22:19 ` Fwd: " Viliam Kubis
2010-07-08  6:33   ` Clemens Ladisch
2010-07-09  1:32     ` Raymond Yau
     [not found]       ` <op.vfkm5fn9y9cctd@localhost>
2010-07-09 11:05         ` Viliam Kubis
     [not found]     ` <op.vfi12awty9cctd@localhost>
2010-07-09 11:04       ` Viliam Kubis [this message]
     [not found]     ` <op.vfi2hgary9cctd@localhost>
2010-07-09 11:05       ` Viliam Kubis
2010-07-12  6:57         ` Clemens Ladisch
2010-07-12 10:58           ` Viliam Kubis
2010-07-22 20:36             ` Simon Schubert
  -- strict thread matches above, loose matches on Subject: below --
2010-07-10  9:41 Raymond Yau
2010-07-10 12:39 ` Viliam Kubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=op.vfko1ulyy9cctd@localhost \
    --to=viliam.kubis@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=clemens@ladisch.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).