* [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
@ 2015-12-15 21:51 Ben Zhang
2015-12-16 19:21 ` Mark Brown
2015-12-16 19:24 ` Applied "ASoC: rt5677: Reconfigure PLL1 after resume" to the asoc tree Mark Brown
0 siblings, 2 replies; 5+ messages in thread
From: Ben Zhang @ 2015-12-15 21:51 UTC (permalink / raw)
To: Mark Brown
Cc: Oder Chiou, alsa-devel, Liam Girdwood, John Lin, Ben Zhang,
Anatol Pomozov, Bard Liao, Dylan Reid
Sometimes PLL1 stops working if the codec loses power
during suspend (when pow-ldo2 or reset gpio is used).
MX-7Bh(RT5677_PLL1_CTRL2) is cleared and won't be restored
by regcache since it's volatile. MX-7Bh has one status bit
and M code for PLL1. rt5677_set_dai_pll doesn't reconfigure
PLL1 after resume because it thinks the PLL params are not
changed.
This patch clears the cached PLL params at resume so that
rt5677_set_dai_pll can reconfigure the PLL after resume.
Signed-off-by: Ben Zhang <benzh@chromium.org>
---
sound/soc/codecs/rt5677.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/codecs/rt5677.c b/sound/soc/codecs/rt5677.c
index f73fd12..c404f51 100644
--- a/sound/soc/codecs/rt5677.c
+++ b/sound/soc/codecs/rt5677.c
@@ -4792,6 +4792,9 @@ static int rt5677_resume(struct snd_soc_codec *codec)
struct rt5677_priv *rt5677 = snd_soc_codec_get_drvdata(codec);
if (!rt5677->dsp_vad_en) {
+ rt5677->pll_src = 0;
+ rt5677->pll_in = 0;
+ rt5677->pll_out = 0;
gpiod_set_value_cansleep(rt5677->pow_ldo2, 1);
gpiod_set_value_cansleep(rt5677->reset_pin, 0);
if (rt5677->pow_ldo2 || rt5677->reset_pin)
--
2.6.0.rc2.230.g3dd15c0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
2015-12-15 21:51 [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume Ben Zhang
@ 2015-12-16 19:21 ` Mark Brown
2015-12-17 5:21 ` Bard Liao
2015-12-16 19:24 ` Applied "ASoC: rt5677: Reconfigure PLL1 after resume" to the asoc tree Mark Brown
1 sibling, 1 reply; 5+ messages in thread
From: Mark Brown @ 2015-12-16 19:21 UTC (permalink / raw)
To: Ben Zhang
Cc: Oder Chiou, alsa-devel, Liam Girdwood, John Lin, Anatol Pomozov,
Bard Liao, Dylan Reid
[-- Attachment #1.1: Type: text/plain, Size: 926 bytes --]
On Tue, Dec 15, 2015 at 01:51:25PM -0800, Ben Zhang wrote:
> Sometimes PLL1 stops working if the codec loses power
> during suspend (when pow-ldo2 or reset gpio is used).
> MX-7Bh(RT5677_PLL1_CTRL2) is cleared and won't be restored
> by regcache since it's volatile. MX-7Bh has one status bit
> and M code for PLL1. rt5677_set_dai_pll doesn't reconfigure
> PLL1 after resume because it thinks the PLL params are not
> changed.
> This patch clears the cached PLL params at resume so that
> rt5677_set_dai_pll can reconfigure the PLL after resume.
This is something I'd expect the machine driver to handle - if it's
stopped using the PLL I'd expect it to explicitly turn it off, suspend
(or the CODEC going idle for that matter normally, the PLL is burning
power) should do that. If anything I'd expect the CODEC driver to
restore the current settings.
That said I'll apply this since it at least doesn't make things worse.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Applied "ASoC: rt5677: Reconfigure PLL1 after resume" to the asoc tree
2015-12-15 21:51 [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume Ben Zhang
2015-12-16 19:21 ` Mark Brown
@ 2015-12-16 19:24 ` Mark Brown
1 sibling, 0 replies; 5+ messages in thread
From: Mark Brown @ 2015-12-16 19:24 UTC (permalink / raw)
To: Ben Zhang, Mark Brown; +Cc: alsa-devel
The patch
ASoC: rt5677: Reconfigure PLL1 after resume
has been applied to the asoc tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
>From 1aa844cd56c7a2b94824f02495ff7ae5d52a7e91 Mon Sep 17 00:00:00 2001
From: Ben Zhang <benzh@chromium.org>
Date: Tue, 15 Dec 2015 13:51:25 -0800
Subject: [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
Sometimes PLL1 stops working if the codec loses power
during suspend (when pow-ldo2 or reset gpio is used).
MX-7Bh(RT5677_PLL1_CTRL2) is cleared and won't be restored
by regcache since it's volatile. MX-7Bh has one status bit
and M code for PLL1. rt5677_set_dai_pll doesn't reconfigure
PLL1 after resume because it thinks the PLL params are not
changed.
This patch clears the cached PLL params at resume so that
rt5677_set_dai_pll can reconfigure the PLL after resume.
Signed-off-by: Ben Zhang <benzh@chromium.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
sound/soc/codecs/rt5677.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/codecs/rt5677.c b/sound/soc/codecs/rt5677.c
index f73fd125e49c..c404f515376e 100644
--- a/sound/soc/codecs/rt5677.c
+++ b/sound/soc/codecs/rt5677.c
@@ -4792,6 +4792,9 @@ static int rt5677_resume(struct snd_soc_codec *codec)
struct rt5677_priv *rt5677 = snd_soc_codec_get_drvdata(codec);
if (!rt5677->dsp_vad_en) {
+ rt5677->pll_src = 0;
+ rt5677->pll_in = 0;
+ rt5677->pll_out = 0;
gpiod_set_value_cansleep(rt5677->pow_ldo2, 1);
gpiod_set_value_cansleep(rt5677->reset_pin, 0);
if (rt5677->pow_ldo2 || rt5677->reset_pin)
--
2.6.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
2015-12-16 19:21 ` Mark Brown
@ 2015-12-17 5:21 ` Bard Liao
2015-12-17 12:12 ` Mark Brown
0 siblings, 1 reply; 5+ messages in thread
From: Bard Liao @ 2015-12-17 5:21 UTC (permalink / raw)
To: Mark Brown, Ben Zhang
Cc: Oder Chiou, alsa-devel, Liam Girdwood, John Lin, Anatol Pomozov,
Dylan Reid
> -----Original Message-----
> From: Mark Brown [mailto:broonie@kernel.org]
> Sent: Thursday, December 17, 2015 3:21 AM
> To: Ben Zhang
> Cc: alsa-devel@alsa-project.org; Liam Girdwood; Bard Liao; Oder Chiou;
> John Lin; Dylan Reid; Anatol Pomozov
> Subject: Re: [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
>
> On Tue, Dec 15, 2015 at 01:51:25PM -0800, Ben Zhang wrote:
> > Sometimes PLL1 stops working if the codec loses power during suspend
> > (when pow-ldo2 or reset gpio is used).
> > MX-7Bh(RT5677_PLL1_CTRL2) is cleared and won't be restored by
> regcache
> > since it's volatile. MX-7Bh has one status bit and M code for PLL1.
> > rt5677_set_dai_pll doesn't reconfigure
> > PLL1 after resume because it thinks the PLL params are not changed.
>
> > This patch clears the cached PLL params at resume so that
> > rt5677_set_dai_pll can reconfigure the PLL after resume.
>
> This is something I'd expect the machine driver to handle - if it's stopped
> using the PLL I'd expect it to explicitly turn it off, suspend (or the CODEC
> going idle for that matter normally, the PLL is burning
> power) should do that. If anything I'd expect the CODEC driver to restore
> the current settings.
The main reason of this patch is MX-7Bh(RT5677_PLL1_CTRL2) is volatile.
However, the status bit of MX-7Bh is currently not used by codec driver.
So, it should be no problem if we set it non-volatile.
>
> That said I'll apply this since it at least doesn't make things worse.
Can we revert it and resend a patch?
>
> ------Please consider the environment before printing this e-mail.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume
2015-12-17 5:21 ` Bard Liao
@ 2015-12-17 12:12 ` Mark Brown
0 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2015-12-17 12:12 UTC (permalink / raw)
To: Bard Liao
Cc: Oder Chiou, alsa-devel, Liam Girdwood, John Lin, Ben Zhang,
Anatol Pomozov, Dylan Reid
[-- Attachment #1.1: Type: text/plain, Size: 716 bytes --]
On Thu, Dec 17, 2015 at 05:21:05AM +0000, Bard Liao wrote:
> > This is something I'd expect the machine driver to handle - if it's stopped
> > using the PLL I'd expect it to explicitly turn it off, suspend (or the CODEC
> > going idle for that matter normally, the PLL is burning
> > power) should do that. If anything I'd expect the CODEC driver to restore
> > the current settings.
> The main reason of this patch is MX-7Bh(RT5677_PLL1_CTRL2) is volatile.
> However, the status bit of MX-7Bh is currently not used by codec driver.
> So, it should be no problem if we set it non-volatile.
> > That said I'll apply this since it at least doesn't make things worse.
> Can we revert it and resend a patch?
Sure.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-12-17 12:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-15 21:51 [PATCH] ASoC: rt5677: Reconfigure PLL1 after resume Ben Zhang
2015-12-16 19:21 ` Mark Brown
2015-12-17 5:21 ` Bard Liao
2015-12-17 12:12 ` Mark Brown
2015-12-16 19:24 ` Applied "ASoC: rt5677: Reconfigure PLL1 after resume" to the asoc tree Mark Brown
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.