diff mbox

ASoC: Intel: set initial runtime PM status to active for ACPI-enumerated ADSP

Message ID 4c86d19eb8db54ac57244ee77e873d5bc22c1b21.1423826462.git.mengdong.lin@intel.com (mailing list archive)
State Accepted
Commit 7bd345c9e87d879d696c6843fe200b60c2051c84
Headers show

Commit Message

Lin, Mengdong Feb. 13, 2015, 11:21 a.m. UTC 
From: Mengdong Lin <mengdong.lin@intel.com>

The ADSP on Braswell/Baytrail is an ACPI device. This patch sets its initial
runtime PM status to active. Otherwise, its initial status is suspended and
runtime_suspend ops will not be called after probe and thus cannot further
trigger ACPI _PS3 (D3) method to put the device into low power D3cold state.

Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>

Comments

Mark Brown Feb. 14, 2015, 4:49 a.m. UTC | #1 
On Fri, Feb 13, 2015 at 07:21:25PM +0800, mengdong.lin@intel.com wrote:
> From: Mengdong Lin <mengdong.lin@intel.com>
> 
> The ADSP on Braswell/Baytrail is an ACPI device. This patch sets its initial
> runtime PM status to active. Otherwise, its initial status is suspended and
> runtime_suspend ops will not be called after probe and thus cannot further
> trigger ACPI _PS3 (D3) method to put the device into low power D3cold state.

Applied, thanks.
diff mbox

Patch

diff --git a/sound/soc/intel/sst/sst.c b/sound/soc/intel/sst/sst.c
index 8a8d56a..d6ea800 100644
--- a/sound/soc/intel/sst/sst.c
+++ b/sound/soc/intel/sst/sst.c
@@ -379,6 +379,10 @@  void sst_configure_runtime_pm(struct intel_sst_drv *ctx)
 	 * initially active. So change the state to active before
 	 * enabling the pm
 	 */
+
+	if (!acpi_disabled)
+		pm_runtime_set_active(ctx->dev);
+
 	pm_runtime_enable(ctx->dev);
 
 	if (acpi_disabled)