[alsa-devel] [PATCH] ASoC: Tegra: Suspend/resume support
ASoC machine drivers that are their own platform_driver (as opposed to those using the soc-audio platform_driver) need to explicitly set up power-management operation callbacks.
Signed-off-by: Stephen Warren swarren@nvidia.com --- Note: This has been tested in the ChromeOS-2.6.37 kernel, with a backport of ASoC for-2.6.39, on Seaboard. I have not tested it in mainline (I'm not sure whether mainline has PM support for the Tegra platform yet), although I did at least compile-test it.
sound/soc/tegra/harmony.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/sound/soc/tegra/harmony.c b/sound/soc/tegra/harmony.c index 8585957..a67d521 100644 --- a/sound/soc/tegra/harmony.c +++ b/sound/soc/tegra/harmony.c @@ -366,10 +366,17 @@ static int __devexit tegra_snd_harmony_remove(struct platform_device *pdev) return 0; }
+const struct dev_pm_ops harmony_pm_ops = { + .suspend = snd_soc_suspend, + .resume = snd_soc_resume, + .poweroff = snd_soc_poweroff, +}; + static struct platform_driver tegra_snd_harmony_driver = { .driver = { .name = DRV_NAME, .owner = THIS_MODULE, + .pm = &harmony_pm_ops, }, .probe = tegra_snd_harmony_probe, .remove = __devexit_p(tegra_snd_harmony_remove),
On Tue, Apr 05, 2011 at 02:27:36PM -0600, Stephen Warren wrote:
+const struct dev_pm_ops harmony_pm_ops = {
- .suspend = snd_soc_suspend,
- .resume = snd_soc_resume,
- .poweroff = snd_soc_poweroff,
+};
We should add an export for snd_soc_pm_ops too so that drivers like this that can just use the default behaviour don't need to cut'n'paste the code like this.
participants (2)
-
Mark Brown
-
Stephen Warren