All of lore.kernel.org
 help / color / mirror / Atom feed
From: bugtrack@alsa-project.org
To: alsa-devel@alsa-project.org
Subject: [ALSA - driver 0001824]: Always mixed into one channel when recording stereo
Date: Mon, 3 Apr 2006 15:38:00 +0200	[thread overview]
Message-ID: <17d9efb69a29e522c046469654faef3e@bugtrack.alsa-project.org> (raw)


A NOTE has been added to this issue.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1824> 
======================================================================
Reported By:                Electro
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   1824
Category:                   PCI - ice1724
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Distribution:               Gentoo
Kernel Version:             2.6.14-gentoo-r4
======================================================================
Date Submitted:             02-06-2006 06:56 CET
Last Modified:              04-03-2006 15:38 CEST
======================================================================
Summary:                    Always mixed into one channel when recording stereo
Description: 
When recording stereo audio (2 channels), the sound seems to mix to one
channel and then saved in the left track. I do not have anything in
asound.conf or ~/.asoundrc that will affect the recordings. I even cleared
them out to make sure that my code is not screwing up the recordings. It
has done this since I upgraded to 1.0.10 from 1.0.9. I thought 1.0.11rc3
will fix the problem but it still does it.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
duplicate of        0001527 amixer fails to set right 'Capture Sour...
======================================================================

----------------------------------------------------------------------
 Electro - 03-27-06 02:54 
----------------------------------------------------------------------
The patch has no effect. It changes the wrong lines, so I remake the patch
myself that corrects the proper lines. While it is compiling, I check the
file to make sure it apply was correct and yes it is correct.

All the debugging and trying to fix it is making the the code for
Audiotrak Prodigy 7.1 worst than before. The whole code for the Audiotrak
Prodigy 7.1 need to be rewritten at a different view point.

I am using alsa-driver-1.0.10-rc2.ebuild which uses alsa-1.0.10. I edit
the ebuild to apply the patch below.

diff -Nru alsa-kernel/pci/ice1712/aureon.c
alsa-kernel.new/pci/ice1712/aureon.c
--- alsa-kernel/pci/ice1712/aureon.c    2005-11-04 06:58:11.000000000
-0700
+++ alsa-kernel.new/pci/ice1712/aureon.c        2006-03-25
22:52:58.867880192 -0700
@@ -1743,7 +1743,7 @@
                0x18, 0x000,            /* All power-up */
 
                0x16, 0x022,            /* I2S, normal polarity, 24bit,
high-pass on */
-               0x17, 0x006,            /* 128fs, slave mode */
+               0x17, 0x022,            /* 256fs, slave mode */
 
                0x00, 0,                /* DAC1 analog mute */
                0x01, 0,                /* DAC2 analog mute */


Until it is fixed, I will just manually specify the cset values using
amixer to record from either AUX, Lin in, Mic, or AC97.

----------------------------------------------------------------------
 ungod - 04-03-06 15:38 
----------------------------------------------------------------------
The patches was made for the CVS version of ALSA (and actually with a few
extra patches) which is why the line numbers were off.

In what way do you mean that the Prodigy driver should be rewritten?
Technically the driver was originally intended for Terratec Aureon cards,
but since the Prodigy 7.1 is practically identical to the Aureon 7.1
Space, support for Prodigy was added. As for the code as a whole, I agree
that it needs alot of cleaning. When I began working on the driver, the
basic structure of the driver was already made, so when I added AC'97
support etc., I followed the same pattern (chaos). Since I am not an
offical ALSA developer or anything, I thought that the ALSA dudes had a
meaning behind the whole, and either way it would be difficult for me to
get en enormous patch accepted.
During the whole process I was careful not to change the code related to
the Prodigy card, as I only had my Aureon Space card to test with. (and
good clear photographs of Aureon Sky and Aureon Universe, but none of
Prodigy) Therefore initialization etc. for the prodigy is practically
untouched although I believe that the Prodigy 7.1 is in fact identical to
the Aureon Space, except for having coax instead of optical in/out.

Anyway... The patch was insufficient? My first go was to make the prodigy
use the aureon WM8770 initialization procedure, but again I was relunctant
to do so, since the developer behind the prodigy support purposeful made a
seperate WM8770 initialization which differed from the aureon one instead
of just reusing it.


I appologize for the delay, but the site refused to insert my note into
the DB.

Issue History
Date Modified  Username       Field                    Change              
======================================================================
02-06-06 06:56 Electro        New Issue                                    
02-06-06 06:56 Electro        Distribution              => Gentoo          
02-06-06 06:56 Electro        Kernel Version            => 2.6.14-gentoo-r4
02-10-06 23:38 tartif         Note Added: 0008009                          
02-16-06 07:26 Electro        Note Added: 0008087                          
02-18-06 01:32 rlrevell       Relationship added       duplicate of 0001527
02-28-06 22:45 Electro        Note Added: 0008248                          
03-20-06 12:35 ungod          Note Added: 0008717                          
03-21-06 09:28 Electro        Note Added: 0008740                          
03-21-06 16:47 ungod          Note Added: 0008743                          
03-21-06 16:48 ungod          Note Edited: 0008743                         
03-25-06 06:53 Electro        Note Added: 0008942                          
03-25-06 20:37 ungod          Note Added: 0008949                          
03-25-06 20:38 ungod          File Added: prodigy-wminit.diff                   

03-25-06 20:44 ungod          Note Added: 0008950                          
03-27-06 02:54 Electro        Note Added: 0008982                          
04-03-06 15:38 ungod          Note Added: 0009098                          
======================================================================




-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642

             reply	other threads:[~2006-04-03 13:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-03 13:38 bugtrack [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-03-27  0:54 [ALSA - driver 0001824]: Always mixed into one channel when recording stereo bugtrack
2006-03-25 19:44 bugtrack
2006-03-25 19:37 bugtrack
2006-03-25  5:53 bugtrack
2006-03-21 15:47 bugtrack
2006-03-21  8:28 bugtrack
2006-03-20 11:35 bugtrack
2006-02-28 21:45 bugtrack
2006-02-18  0:32 bugtrack
2006-02-16  6:26 bugtrack
2006-02-10 22:38 bugtrack
2006-02-06  5:56 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=17d9efb69a29e522c046469654faef3e@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 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.