All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.