From: Takashi Iwai <tiwai@suse.de>
To: Apostolos Dimitromanolakis <apostolos@aei.ca>
Cc: Alsa-devel <alsa-devel@lists.sourceforge.net>
Subject: Re: Prodigy 7.1 driver update [snd-ice1724]
Date: Mon, 24 Nov 2003 19:33:34 +0100 [thread overview]
Message-ID: <s5hptfh1url.wl@alsa2.suse.de> (raw)
In-Reply-To: <3FC245AB.8040902@aei.ca>
At Mon, 24 Nov 2003 12:53:47 -0500,
Apostolos Dimitromanolakis wrote:
>
> Hi,
>
> here's the new revision of the Audiotrak Prodigy 7.1 driver. The
> driver now initializes correctly for the 24.576Mhz crystal and also I
> have added some more controls for adc oversampling ratio (in theory
> should be changed automatically to 64x at 96Khz sampling rate) and dac
> de-emphasis.
>
> The initialization sequence of the WM8770 has been modified to reduce
> the pop noise at startup (now it's almost nono-existant). Also the
> volume control updates now only at zero cross automatically to reduce
> again the noise when changing volumes. I think the aureon driver should
> also benefit from these two so I include a patch for the aureon.c too.
thanks, applied to cvs with minor modifications.
i changed the put callback to compare with the old value to avoid the
no necessary performance. too often i2c transfer can be problematic
when a crazy mixer apps accesses to mixer elements.
> Finally there is some misconception in the aureon & prodigy drivers:
> the codec chip does not have a Master control as it appears at the
> mixer. Instead the master control just updates the 8 volume controls of
> the individual DAC at the same time. Probably the best way is to add a
> virtual master control (ie that we update the volumes of the dac with
> <master attenuation> + <specific channel attenuation> ).
well, such a thing should be done on user-space rather than on
kernel. let's postpone this until the new highlevel mixer functions
become available.
>
> As I wrote some days ago, the default buffer/period rates that
> alsarecord uses at 44.1KHz and lower sampling , locks hard the kernel.
> Have no idea what is it/how to fix it whatsoever.
hmm, could be the mismatch of cristal rate?
i don't find out the critical path in the driver yet.
Takashi
-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive? Does it
help you create better code? SHARE THE LOVE, and help us help
YOU! Click Here: http://sourceforge.net/donate/
next prev parent reply other threads:[~2003-11-24 18:33 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-24 17:53 Prodigy 7.1 driver update [snd-ice1724] Apostolos Dimitromanolakis
2003-11-24 18:33 ` Takashi Iwai [this message]
2003-11-25 19:38 ` Apostolos Dimitromanolakis
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=s5hptfh1url.wl@alsa2.suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@lists.sourceforge.net \
--cc=apostolos@aei.ca \
/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.