diff options
| author | Liam Girdwood <liam.r.girdwood@linux.intel.com> | 2015-05-28 08:14:18 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-05-28 21:27:00 +0200 |
| commit | 01f202c7b4b40025f3ea4721c52e7f78545e3b07 (patch) | |
| tree | 8732a6c7b99718003a482a69dbbd452c4fdfe785 /sound/soc/intel/haswell | |
| parent | ASoC: Intel: remove unused function hsw_pcm_free_modules() (diff) | |
| download | linux-01f202c7b4b40025f3ea4721c52e7f78545e3b07.tar.xz linux-01f202c7b4b40025f3ea4721c52e7f78545e3b07.zip | |
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 225c04c38e42..1557e37abe19 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) |