All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: "Babu, Ramesh" <ramesh.babu@intel.com>
Cc: Peter Ujfalusi <peter.ujfalusi@ti.com>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Liam Girdwood <lrg@ti.com>,
	Misael Lopez Cruz <misael.lopez@ti.com>
Subject: Re: [PATCH v2 1/2] ASoC: core: Add flag to ignore pmdown_time at pcm_close
Date: Fri, 21 Oct 2011 09:53:04 +0100	[thread overview]
Message-ID: <20111021085303.GB3513@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <12D0C12AF19E15409D57F22566E88EF50A813690AD@bgsmsx501.gar.corp.intel.com>

On Fri, Oct 21, 2011 at 11:42:27AM +0530, Babu, Ramesh wrote:

> I feel ignore_pmdown_time needs to be part of snd_soc_dai_driver structure.
> It will give flexibility to control individual dai.  Individual dai's can choose to
> Ignore power down delay.

Why do you need to do this?

> Would it make sense?

> In my codec, I need to ignore the power down delay for voice dais, but retain the delay
> for HiFi dai's.

I'm not sure associating it with DAIs is a good idea, I'd expect any
issues to be with the analouge outputs not DAIs.

  reply	other threads:[~2011-10-21  8:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-14 11:43 [PATCH v2 0/2] ASoC: core: Ignoring pmdown_time for playback stream power down Peter Ujfalusi
2011-10-14 11:43 ` [PATCH v2 1/2] ASoC: core: Add flag to ignore pmdown_time at pcm_close Peter Ujfalusi
2011-10-21  6:12   ` Babu, Ramesh
2011-10-21  8:53     ` Mark Brown [this message]
2011-10-21  9:10       ` Babu, Ramesh
2011-10-21  9:13         ` Mark Brown
2011-10-21 10:58           ` Babu, Ramesh
2011-10-21 11:26             ` Vinod Koul
2011-10-21 17:01             ` Nallasellan, Singaravelan
2011-10-21 19:41               ` Mark Brown
2011-10-21 20:24                 ` Nallasellan, Singaravelan
2011-10-22  9:33                   ` Mark Brown
2011-10-22  9:45             ` Mark Brown
2011-10-22 17:17               ` Vinod Koul
2011-10-23 14:00                 ` Mark Brown
2011-10-24  2:55                   ` Vinod Koul
2011-10-14 11:43 ` [PATCH v2 2/2] ASoC: twl6040: Request core to inline the DAPM sequence Peter Ujfalusi
2011-10-14 19:44 ` [PATCH v2 0/2] ASoC: core: Ignoring pmdown_time for playback stream power down 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=20111021085303.GB3513@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=lrg@ti.com \
    --cc=misael.lopez@ti.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=ramesh.babu@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.