All of lore.kernel.org
 help / color / mirror / Atom feed
From: noreply@alsa-project.org
To: alsa-devel@alsa-project.org
Subject: [ALSA - driver 0000320]: FL is FL+FR and FR is FR+FL
Date: Thu, 1 Jul 2004 09:43:53 +0200	[thread overview]
Message-ID: <cf478d736da6bfde6134bb73874f28b6@bugtrack.alsa-project.org> (raw)


The following bug has been RESOLVED.
======================================================================
https://bugtrack.alsa-project.org/alsa-bug/bug_view_advanced_page.php?bug_id=0000320
======================================================================
Reported By:                Voluspa
Assigned To:                pzad
======================================================================
Project:                    ALSA - driver
Bug ID:                     320
Category:                   PCI - emu10k1
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     resolved
Distribution:               Homebrew
Kernel Version:             2.6.6
Resolution:                 not a bug
======================================================================
Date Submitted:             06-01-2004 10:12 CEST
Last Modified:              07-01-2004 09:43 CEST
======================================================================
Summary:                    FL is FL+FR and FR is FR+FL
Description: 
Card: Sound Blaster Live!
Chip: SigmaTel STAC9758/59

Between 1.0.5rc1 and 1.0.5a the Front Left and Front Right signals are
combined, which creates real crappy sound:

Front Left speaker = Front Left + Front Right
Front Right speaker = Front Right + Front Left

Checking the new "Surround Mix" in alsamixer gets even more confusing:

Front Left speaker = Front Left + Front Right + Center
Front Right speaker = Front Right + Front Left + LFE

There are no differences between the 1.0.5rc1 and 1.0.5a in the
proc/asound/card0/codec97#0/ac97#0-0+regs

Btw, the alsamixer options are getting really out of hand...
Eg a new option called just "Front" (I keep it off since I can't hear any
difference).

Mvh
Mats Johannesson

======================================================================

----------------------------------------------------------------------
 pzad - 06-08-2004 08:40 CEST 
----------------------------------------------------------------------
I think this AC97 codec problem.

----------------------------------------------------------------------
 pzad - 06-23-2004 15:21 CEST 
----------------------------------------------------------------------
Could you attach asound.state or amixer output.
And I think, that ac97 register were changed, but there are not in proc
output, because these are in diffrent pages.

----------------------------------------------------------------------
 Voluspa - 06-24-2004 10:48 CEST 
----------------------------------------------------------------------
OK, rechecked and the /proc-registers are set the same in 1.0.5rc1 and
1.0.5a. Installed 1.0.5a without loading the rc1 asound.state, and
handcrafted a new with alsamixer. Having set all options equal to rc1, the
asound.state file indeed shows some differences.

Uploading asound.state-1.0.5rc1, asound.state-1.0.5a and the diff
asound.state-diff-1.0.5rc1-1.0.5a

----------------------------------------------------------------------
 pzad - 06-30-2004 09:00 CEST 
----------------------------------------------------------------------
Could you try this patch ? It is already in alsa CVS.

http://sourceforge.net/mailarchive/forum.php?thread_id=5027384&forum_id=33141

----------------------------------------------------------------------
 Voluspa - 06-30-2004 15:36 CEST 
----------------------------------------------------------------------
I'm glad that the code review you did amounted to _some_ kind of bugfix
after
me having sent you on a wild goose chase, Peter. This bug report can be
closed, 'cos it's (probably) no bug at all. Sorry... Here's what
happened:

During the development of the STAC9758/59, I kept alsa-driver-1.0.4
updated
by patching in the relevant file changes (no CVS pulls, just manual
changes).
Last date was May 14. The _perceived_ breakage happened between that date
and
1.0.5rc1 - not 1.0.5a as I believed.

1.0.4 + patches had /proc/asound/card0/codec97#0/ac97#0-0+regs
[...]
0:20 = 2000
0:22 = 0000

The bit in register 20 means "3D EFFECT ENABLED", but the "33" value I
had
set with the alsamixer tool didn't take effect (would show up as 0004 in
register 22).

1.0.5rc1 had /proc/asound/card0/codec97#0/ac97#0-0+regs
[...]
0:20 = 2000
0:22 = 0004

And we see the "LINE_OUT SEPARATION RATIO; (LOW)" of "3D stereo
enhancement
function, SigmaTel Surround 3D (SS3D), built into the AC'97 component".

If mixing Front Left and Front Right signals is part of how a 3D effect
is
achieved, then this is no bug. But using a 5.1 speaker system I can only
say
that the effect sounds terrible... So, in the future I keep "3D Control -
Switch [Off]".

Now using alsa-driver-1.0.5a plus the CVS patch you referred to, and all's
well.

Once again, sorry about the fuzz.

Bug History
Date Modified  Username       Field                    Change              
======================================================================
06-01-04 10:12 Voluspa        New Bug                                      
06-01-04 10:12 Voluspa        Distribution              => Homebrew        
06-01-04 10:12 Voluspa        Kernel Version            => 2.6.6           
06-08-04 08:40 pzad           Bugnote Added: 0001294                       
06-23-04 15:21 pzad           Bugnote Added: 0001335                       
06-24-04 10:47 Voluspa        File Added: asound.state-1.0.5rc1                 
  
06-24-04 10:48 Voluspa        Bugnote Added: 0001336                       
06-24-04 10:48 Voluspa        File Added: asound.state-1.0.5a                   

06-24-04 10:49 Voluspa        File Added: asound.state-diff-1.0.5rc1-1.0.5a     
              
06-28-04 09:14 pzad           Assigned To               => pzad            
06-28-04 09:14 pzad           Status                   new => assigned     
06-30-04 09:00 pzad           Bugnote Added: 0001348                       
06-30-04 15:36 Voluspa        Bugnote Added: 0001349                       
07-01-04 09:43 pzad           Resolution               open => not a bug   
07-01-04 09:43 pzad           Status                   assigned => resolved
======================================================================




-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 -
digital self defense, top technical experts, no vendor pitches,
unmatched networking opportunities. Visit www.blackhat.com

             reply	other threads:[~2004-07-01  7:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-01  7:43 noreply [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-09-24 11:32 [ALSA - driver 0000320]: FL is FL+FR and FR is FR+FL bugtrack
2004-06-30 13:36 noreply
2004-06-30  7:00 noreply
2004-06-28  7:14 noreply
2004-06-24  8:48 noreply
2004-06-23 13:21 noreply
2004-06-08  6:40 noreply

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=cf478d736da6bfde6134bb73874f28b6@bugtrack.alsa-project.org \
    --to=noreply@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.