All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Randy Cushman <rcushman_linux@earthlink.net>
Cc: Alsa-devel@lists.sourceforge.net
Subject: Re: Patch for AD1986 AC97 CODEC
Date: Fri, 22 Dec 2006 13:12:21 +0100	[thread overview]
Message-ID: <s5htzzoqgd6.wl%tiwai@suse.de> (raw)
In-Reply-To: <458AD2AB.70602@earthlink.net>

At Thu, 21 Dec 2006 13:30:03 -0500,
Randy Cushman wrote:
> 
> Takashi,
> 
> Attached please find the third and final installment of separated 
> patches you requested.

Thanks.  Now I applied it to ALSA HG tree.

> 
> I have retested all 3 patches applied to ALSA 1.0.13.  (The patches were 
> derived sequentially from Mercurial.)
> 
> I decided to leave out the output volume control renaming for the 
> AD1986, because 1) I found the name "Master Surround" to be confusing, 
> since "Surround" is used to refer to the rear channels of a multichannel 
> configuration.  Also renaming "Headphone" to "Master" makes no sense for 
> the AD1986, since the AD1986 has separate Headphone pins, unlike the 
> AD1888, AD1980 and AD1985 for which the same pins are used for either 
> the Headphone or Surround outputs.
> 
> I do plan to revisit output volume control names in the future.  The 
> AD1986 uses volume controls somewhat differently than most AC '97 
> Devices.  For example, normally for AC '97 devices, register 0x18 
> controls DAC PCM attenuation for all channels; the AD1986 uses a special 
> "ADI Mode" configuration in which register 0x18 controls DAC PCM 
> attenuation only for the front channels, with other registers 
> controlling DAC PCM attenuation for the other channels.
> 
> The AD1888, AD1980 and AD1985 can be configured in software to use 
> either the AC '97 Compatibility mode or the ADI Mode.  (In ALSA 
> currently they are fixed in ADI Mode.)  The AD1986 can use only the ADI 
> Mode.

Yes, these volume controls are confusing indeed...


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

      reply	other threads:[~2006-12-22 12:12 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-21 18:30 Patch for AD1986 AC97 CODEC Randy Cushman
2006-12-22 12:12 ` Takashi Iwai [this message]

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=s5htzzoqgd6.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=Alsa-devel@lists.sourceforge.net \
    --cc=rcushman_linux@earthlink.net \
    /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.