diff options
| author | Liam Girdwood <liam.r.girdwood@linux.intel.com> | 2015-05-28 06:14:18 (GMT) |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-05-28 19:27:00 (GMT) |
| commit | 01f202c7b4b40025f3ea4721c52e7f78545e3b07 (patch) | |
| tree | 8732a6c7b99718003a482a69dbbd452c4fdfe785 /sound/soc/intel/haswell | |
| parent | 506c148ee5e1bfb836116353305927ca4c21a23e (diff) | |
| download | linux-01f202c7b4b40025f3ea4721c52e7f78545e3b07.tar.xz | |
ASoC: Intel: fix broadwell module removing failed issue
In haswell-pcm module unloading, we can't free runtime modules
directly, for they may be already freed in runtime suspend.
Here add executing suspend call to unload runtime modules, only
for status not equal to RPM_SUSPEND, to fix broadwell module
removing failed issue.
Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>
Signed-off-by: Jie Yang <yang.jie@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/intel/haswell')
| -rw-r--r-- | sound/soc/intel/haswell/sst-haswell-pcm.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sound/soc/intel/haswell/sst-haswell-pcm.c b/sound/soc/intel/haswell/sst-haswell-pcm.c index 225c04c..1557e37 100644 --- a/sound/soc/intel/haswell/sst-haswell-pcm.c +++ b/sound/soc/intel/haswell/sst-haswell-pcm.c @@ -1103,8 +1103,10 @@ static int hsw_pcm_remove(struct snd_soc_platform *platform) snd_soc_platform_get_drvdata(platform); int i; + /* execute a suspend call to unload all FW resources */ + if (!pm_runtime_status_suspended(platform->dev)) + pm_runtime_put_sync_suspend(platform->dev); pm_runtime_disable(platform->dev); - hsw_pcm_free_modules(priv_data); for (i = 0; i < ARRAY_SIZE(hsw_dais); i++) { if (hsw_dais[i].playback.channels_min) |