From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl>
Cc: alsa-devel@alsa-project.org, Tony Lindgren <tony@atomide.com>,
linux-kernel@vger.kernel.org,
Jarkko Nikula <jarkko.nikula@bitmer.com>,
linux-omap@vger.kernel.org, Liam Girdwood <lrg@ti.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/4] ASoC: cx20442: add bias control over a platform provided regulator
Date: Tue, 27 Dec 2011 17:26:00 +0000 [thread overview]
Message-ID: <20111227172559.GB22970@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <201112271516.09114.jkrzyszt@tis.icnet.pl>
On Tue, Dec 27, 2011 at 03:16:08PM +0100, Janusz Krzysztofik wrote:
> On Monday 26 of December 2011 at 12:02:01, Mark Brown wrote:
> > which would look a lot more natural - you shouldn't need to remember if
> > the regulator is enabled, you should just turn it on in the STANDBY to
> > PREPARE transition and turn it off in the ON to PREPARE or PREPARE to
> > STANDBY transitions.
> Can I assume STANDBY or OFF at the time the codec .remove method is
> called? If not, is there a helper function available which can be called
> in order to perform all those ON -> PREPARE -> STANDBY [-> OFF] transitions before calling regulator_put()?
It'll be in STANDBY or OFF depending on if the device sets
idle_bias_off.
WARNING: multiple messages have this Message-ID (diff)
From: broonie@opensource.wolfsonmicro.com (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [alsa-devel] [PATCH 2/4] ASoC: cx20442: add bias control over a platform provided regulator
Date: Tue, 27 Dec 2011 17:26:00 +0000 [thread overview]
Message-ID: <20111227172559.GB22970@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <201112271516.09114.jkrzyszt@tis.icnet.pl>
On Tue, Dec 27, 2011 at 03:16:08PM +0100, Janusz Krzysztofik wrote:
> On Monday 26 of December 2011 at 12:02:01, Mark Brown wrote:
> > which would look a lot more natural - you shouldn't need to remember if
> > the regulator is enabled, you should just turn it on in the STANDBY to
> > PREPARE transition and turn it off in the ON to PREPARE or PREPARE to
> > STANDBY transitions.
> Can I assume STANDBY or OFF at the time the codec .remove method is
> called? If not, is there a helper function available which can be called
> in order to perform all those ON -> PREPARE -> STANDBY [-> OFF] transitions before calling regulator_put()?
It'll be in STANDBY or OFF depending on if the device sets
idle_bias_off.
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl>
Cc: alsa-devel@alsa-project.org, Tony Lindgren <tony@atomide.com>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
Liam Girdwood <lrg@ti.com>,
Jarkko Nikula <jarkko.nikula@bitmer.com>
Subject: Re: [alsa-devel] [PATCH 2/4] ASoC: cx20442: add bias control over a platform provided regulator
Date: Tue, 27 Dec 2011 17:26:00 +0000 [thread overview]
Message-ID: <20111227172559.GB22970@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <201112271516.09114.jkrzyszt@tis.icnet.pl>
On Tue, Dec 27, 2011 at 03:16:08PM +0100, Janusz Krzysztofik wrote:
> On Monday 26 of December 2011 at 12:02:01, Mark Brown wrote:
> > which would look a lot more natural - you shouldn't need to remember if
> > the regulator is enabled, you should just turn it on in the STANDBY to
> > PREPARE transition and turn it off in the ON to PREPARE or PREPARE to
> > STANDBY transitions.
> Can I assume STANDBY or OFF at the time the codec .remove method is
> called? If not, is there a helper function available which can be called
> in order to perform all those ON -> PREPARE -> STANDBY [-> OFF] transitions before calling regulator_put()?
It'll be in STANDBY or OFF depending on if the device sets
idle_bias_off.
next prev parent reply other threads:[~2011-12-27 17:26 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-23 23:12 [PATCH 0/4] Amstrad Delta: access MODEM_RESET GPIO pin over a regulator Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` [PATCH 1/4] ARM: OMAP1: ams-delta: set up regulator over modem reset GPIO pin Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` [PATCH 2/4] ASoC: cx20442: add bias control over a platform provided regulator Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-26 11:02 ` Mark Brown
2011-12-26 11:02 ` Mark Brown
2011-12-26 11:02 ` Mark Brown
2011-12-26 12:18 ` Janusz Krzysztofik
2011-12-26 12:18 ` [alsa-devel] " Janusz Krzysztofik
2011-12-26 12:18 ` Janusz Krzysztofik
2011-12-27 14:16 ` Janusz Krzysztofik
2011-12-27 14:16 ` [alsa-devel] " Janusz Krzysztofik
2011-12-27 14:16 ` Janusz Krzysztofik
2011-12-27 17:26 ` Mark Brown [this message]
2011-12-27 17:26 ` Mark Brown
2011-12-27 17:26 ` Mark Brown
2011-12-23 23:12 ` [PATCH 3/4] ARM: OMAP1: ams-delta: update the modem to use regulator API Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-23 23:12 ` [PATCH 4/4] ASoC: OMAP: ams-delta: drop .set_bias_level callback Janusz Krzysztofik
2011-12-23 23:12 ` Janusz Krzysztofik
2011-12-26 11:03 ` Mark Brown
2011-12-26 11:03 ` Mark Brown
2011-12-26 11:03 ` Mark Brown
2011-12-26 12:30 ` Janusz Krzysztofik
2011-12-26 12:30 ` Janusz Krzysztofik
2011-12-26 12:30 ` Janusz Krzysztofik
2011-12-26 17:23 ` Mark Brown
2011-12-26 17:23 ` Mark Brown
2011-12-26 20:43 ` Janusz Krzysztofik
2011-12-26 20:43 ` Janusz Krzysztofik
2011-12-26 20:43 ` Janusz Krzysztofik
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=20111227172559.GB22970@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=jarkko.nikula@bitmer.com \
--cc=jkrzyszt@tis.icnet.pl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=lrg@ti.com \
--cc=tony@atomide.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.