[alsa-devel] [PATCH] ASoC: omap-mcbsp: Add pm_runtime_get/put functions call for McBSP.
![](https://secure.gravatar.com/avatar/32ec4b8fb9a61cd6fdace1bc30d04235.jpg?s=120&d=mm&r=g)
pm_runtime_get_sync() and put_sync() are not called by McBSP driver. This is introducing a problem with PM and Audio Backend due to missing get/put for McBSP IP.
Signed-off-by: Sebastien Guiriec s-guiriec@ti.com --- sound/soc/omap/omap-mcbsp.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c index 1046083..007d239 100644 --- a/sound/soc/omap/omap-mcbsp.c +++ b/sound/soc/omap/omap-mcbsp.c @@ -117,6 +117,8 @@ static int omap_mcbsp_dai_startup(struct snd_pcm_substream *substream, struct omap_mcbsp *mcbsp = snd_soc_dai_get_drvdata(cpu_dai); int err = 0;
+ pm_runtime_get_sync(mcbsp->dev); + if (!cpu_dai->active) err = omap_mcbsp_request(mcbsp);
@@ -165,6 +167,8 @@ static void omap_mcbsp_dai_shutdown(struct snd_pcm_substream *substream, omap_mcbsp_free(mcbsp); mcbsp->configured = 0; } + + pm_runtime_put_sync(mcbsp->dev); }
static int omap_mcbsp_dai_trigger(struct snd_pcm_substream *substream, int cmd,
![](https://secure.gravatar.com/avatar/d28dfe03ea754ea1153719f4ced12649.jpg?s=120&d=mm&r=g)
On Fri, Jun 01, 2012 at 10:12:52PM +0200, Sebastien Guiriec wrote:
pm_runtime_get_sync() and put_sync() are not called by McBSP driver. This is introducing a problem with PM and Audio Backend due to missing get/put for McBSP IP.
They are called by the core. Probably you're just pointing your struct device at some virtual device instead of the physical device, either enabling runtime PM for the virtual device (assuming it's a child of the physical device) or changing the pointer to the physical device (which might make for better dev_ logging potentially) should resolve the issue.
pm_runtime_put_sync(mcbsp->dev);
No need for this, anyway - just use regular _put(). We don't care if the device is actually suspended before we return, the PM core can do that at it's leisure.
participants (2)
-
Mark Brown
-
Sebastien Guiriec