public inbox for alsa-devel@alsa-project.org
 help / color / mirror / Atom feed
From: bugtrack@alsa-project.org
To: alsa-devel@alsa-project.org
Subject: [ALSA - driver 0001731]: Fixed chip configuration for 5.1 in ALC850 code
Date: Wed, 11 Jan 2006 11:54:07 +0100	[thread overview]
Message-ID: <c4f9685403f0af2d18def4e0718f273a@bugtrack.alsa-project.org> (raw)


A NOTE has been added to this issue.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1731> 
======================================================================
Reported By:                C-MoH
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   1731
Category:                   PCI - intel8x0
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Distribution:               
Kernel Version:             
======================================================================
Date Submitted:             01-11-2006 01:17 CET
Last Modified:              01-11-2006 11:54 CET
======================================================================
Summary:                    Fixed chip configuration for 5.1 in ALC850 code
Description: 
Frustrated by the fact that the 5.1 DVD I just bought had no voices (since
they were on the center channel) I started to dig into it. The details are
in the "steps to reproduce".

Attached is a (very) small patch that fixed it completely on my system
(both the mixer and the channel mix-up). My guess is that the chip was
misconfigured by lack of a bit in the MX76 register. (2 instead of 5 for
bits 6:4)
======================================================================

----------------------------------------------------------------------
 rlrevell - 01-11-06 01:40 
----------------------------------------------------------------------
Unfortunately it only appears in the advanced view, I don't like this
either.

So you are saying you regression tested your patch fully?

----------------------------------------------------------------------
 C-MoH - 01-11-06 11:54 
----------------------------------------------------------------------
I only tested the patch superficially with speaker-test and the DVD which
brought me to this.

What I did and worked fine:
speaker-test -Dsurround51 -c6
speaker-test -Dsurround40 -c4
speaker-test -c2

For the 5.1 DVD everything was now fine (mixer, sound quality, voice
etc.):
mplayer -ao alsa -channels 6 dvd://

Also with alsamixer the mixer channels (center and lfe) now do have effect
over the volume/muting of sound. The behavior of switches is no longer
erratical (i.e. the IEC958/Mix/Mono Mix/Video/DuplicateFront had a weird
interaction, which now seemed to have a more normal aspect - in fact I
never thought much of it, but I remarked it now).

All this made me think it's all right. If you take the time to look at the
patch and at the ALC850 specs from Realtek you'll see that... well,
there's not much to verify it's correctness.

If there's more to do I'll do it and post it here. However I thought that
such a big project has its own testing procedures - remember I'm not a dev
here.

Issue History
Date Modified  Username       Field                    Change              
======================================================================
01-11-06 01:17 C-MoH          New Issue                                    
01-11-06 01:17 C-MoH          File Added: cipi.patch                       
01-11-06 01:21 C-MoH          Issue Monitored: C-MoH                       
01-11-06 01:22 C-MoH          Note Added: 0007553                          
01-11-06 01:33 rlrevell       Note Added: 0007554                          
01-11-06 01:37 C-MoH          Note Added: 0007555                          
01-11-06 01:40 rlrevell       Note Added: 0007556                          
01-11-06 11:54 C-MoH          Note Added: 0007562                          
======================================================================




-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click

             reply	other threads:[~2006-01-11 10:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-11 10:54 bugtrack [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-01-12 11:46 [ALSA - driver 0001731]: Fixed chip configuration for 5.1 in ALC850 code bugtrack
2006-01-12  0:22 bugtrack
2006-01-11 17:38 bugtrack
2006-01-11  0:40 bugtrack
2006-01-11  0:37 bugtrack
2006-01-11  0:33 bugtrack
2006-01-11  0:22 bugtrack
2006-01-11  0:17 bugtrack

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=c4f9685403f0af2d18def4e0718f273a@bugtrack.alsa-project.org \
    --to=bugtrack@alsa-project.org \
    --cc=alsa-devel@alsa-project.org \
    /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