From: Apostolos Dimitromanolakis <apostolos@aei.ca>
To: Takashi Iwai <tiwai@suse.de>, alsa-devel@lists.sourceforge.net
Subject: Re: [PATCH] aureon.c - adc mux, master vol
Date: Thu, 15 Jul 2004 13:40:49 -0400 [thread overview]
Message-ID: <40F6C1A1.5000803@aei.ca> (raw)
In-Reply-To: <s5h4qo98ell.wl@alsa2.suse.de>
Yeah you are right about the master volume as it is implemented but I
think there are seperate analog & digital volume control so it could be
possible to use analog for the master and the digital for the DAC channels.
I will be away from my computer so I can't test the patch right now but
I will have a look at the driver again soon.
Apostolos
Takashi Iwai wrote:
>Hi,
>
>it seems that my last post didn't go out properly...
>
>At Tue, 06 Jul 2004 21:05:44 -0400,
>Apostolos Dimitromanolakis wrote:
>
>
>>I think it would be convenient for the end user to keep the Master
>>volume attenuation on Aureon & Prodigy. There are seperate analog and
>>digital volume controls on the chip and since each DAC has it's analog
>>volume control as a mixer element, why not have the master digital
>>volume as another control?
>>
>>
>
>The master volume control on WM codec seems not to be a real 'master
>control'. It's no independent control but just modifies the all DAC
>volumes at once.
>
>If we keep this, the DAC volumes must be updated and the changes must
>be notified. But then the question is how to handle master volume
>when only a DAC volume is changed?
>
>
>BTW, the attached patch is to use more intuitive control names for
>volume and capture controls. The bogus master volume is removed.
>Could aureon/prodigy owners can test it?
>
>It's to the latest CVS.
>
>
>thanks,
>
>Takashi
>
>
-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=4721&alloc_id=10040&op=click
next prev parent reply other threads:[~2004-07-15 17:40 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-03 15:46 [PATCH] aureon.c - adc mux, master vol Christoph Haderer
2004-07-05 10:00 ` Takashi Iwai
2004-07-07 1:05 ` Apostolos Dimitromanolakis
2004-07-15 15:39 ` Takashi Iwai
2004-07-15 17:40 ` Apostolos Dimitromanolakis [this message]
2004-07-16 9:43 ` Takashi Iwai
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=40F6C1A1.5000803@aei.ca \
--to=apostolos@aei.ca \
--cc=alsa-devel@lists.sourceforge.net \
--cc=tiwai@suse.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 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.