All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: "Koul, Vinod" <vinod.koul@intel.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	"Harsha, Priya" <priya.harsha@intel.com>,
	Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: asoc SND_SOC_DAPM_AIF_IN question
Date: Thu, 23 Dec 2010 11:28:28 +0000	[thread overview]
Message-ID: <20101223112828.GA14481@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <438BB0150E931F4B9CE701519A44630104C104C329@bgsmsx502.gar.corp.intel.com>

On Thu, Dec 23, 2010 at 08:36:07AM +0530, Koul, Vinod wrote:
> On Wed, Dec 22, 2010 at 11:25:29PM +0530, Mark Brown wrote:

> > These "Rail" widgets look quite unusual - what are they?

> Audio Supply rails, I have one global audio rail which controls most of audio 
> widgets and then separate rails for speakers and headset. For example I need to 
> turn on Audio supply rail and then headset rail for headsets to work properly. 
> So I described them as SND_SOC_DAPM_SUPPLY. Added events so that I power them on 
> and power off when stream becomes active or inactive.

"Audio Supply" sounds like the main analogue bias for the CODEC - that
would normally be managed by set_bias_level() rather than by having it
supply every single widget in the CODEC map.

> > What is a "PCM button" and what do you mean by starting a "stream"?
> > Normally a stream would be a path between the CPU and the CODEC.

> This is PCM interface enable between codec and cpu. So for PCM port to work 
> between codec and cpu, we need to enable the PCM port, this one does 
> that. This is going to be a global switch to be turned on for all streams (I 
> have 4 playback and 1 capture, and can be simultaneous).

If you have multiple links between the CPU and the CODEC with a single
power bit to control them all I'd suggest defining several AIF widgets,
each with no power management, then making the actual power controlled
by a supply widget.  That way the power will be enabled as required but
you won't have tied all the data streams together in the DAPM map.

> Here I was trying 
> 1) Putting the PCM_In Button as SND_SOC_DAPM_AIF_IN()

I'm still not sure what you mean by "Button"?  I'm guessing it's a
register control for power.

  reply	other threads:[~2010-12-23 11:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-22 17:09 asoc SND_SOC_DAPM_AIF_IN question Koul, Vinod
2010-12-22 17:55 ` Mark Brown
2010-12-23  3:06   ` Koul, Vinod
2010-12-23 11:28     ` Mark Brown [this message]
2010-12-23 13:01       ` Koul, Vinod
2010-12-24 11:35         ` Mark Brown

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=20101223112828.GA14481@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=lrg@slimlogic.co.uk \
    --cc=priya.harsha@intel.com \
    --cc=vinod.koul@intel.com \
    /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.