From: Liam Girdwood <lrg@slimlogic.co.uk>
To: Peter Ujfalusi <peter.ujfalusi@nokia.com>
Cc: alsa-devel@alsa-project.org,
Mark Brown <broonie@opensource.wolfsonmicro.com>
Subject: Re: [PATCH v2] ASoC: tlv320dac33: Power down digital parts, when not needed
Date: Fri, 10 Dec 2010 22:51:03 +0000 [thread overview]
Message-ID: <1292021463.3422.19.camel@odin> (raw)
In-Reply-To: <1291980391-5202-1-git-send-email-peter.ujfalusi@nokia.com>
On Fri, 2010-12-10 at 13:26 +0200, Peter Ujfalusi wrote:
> If the following scenario has been followed:
> 1. Enable analog bypass
> amixer sset 'Analog Left Bypass' on
> amixer sset 'Analog Right Bypass' on
>
> 2. Start playback
> aplay -fdat -d3 /dev/zero
>
> After the playback stopped (3 sec), and the soc timeout (5 sec),
> the digital parts of the codec will remain powered up.
> This means that the DAI clocks are continue to run, the
> oscillator remain operational, etc.
>
> Use the SND_SOC_DAPM_POST_PMD widget to get notification
> about the stopped stream, and power down the digital
> part of the codec.
> If the analog bypass is enabled, than the codec will remain in
> BIAS_ON level, and things will work correctly.
> In case, if the bypass is disabled, than the codec will
> fall to BIAS_STANDBY than to BIAS_OFF level, as it used
> to.
>
> The digital part of DAC33 is initialized at every stream start
> (DAPM_PRE:PRE_PMU event), so subsequent streams (within 5 sec)
> will have working DAI.
> When the codec is coming out from BIAS_OFF, the full power-up
> sequence followed by the same DAPM_PRE widget event will power up
> the digital part.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
> Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> ---
Applied.
Thanks
Liam
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
prev parent reply other threads:[~2010-12-10 22:51 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-10 11:26 [PATCH v2] ASoC: tlv320dac33: Power down digital parts, when not needed Peter Ujfalusi
2010-12-10 22:51 ` Liam Girdwood [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=1292021463.3422.19.camel@odin \
--to=lrg@slimlogic.co.uk \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=peter.ujfalusi@nokia.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.