From: Jie Yang Date: Sat, 30 May 2015 14:33:57 +0000 (+0800) Subject: ASoC: intel: Revert "ASoC: Intel: fix broadwell module removing failed issue" X-Git-Tag: v5.15~15086^2~80^2~6^2~8 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6e5132f79a2e441bde4818abdc813859c8064901;p=platform%2Fkernel%2Flinux-starfive.git ASoC: intel: Revert "ASoC: Intel: fix broadwell module removing failed issue" This reverts commit 01f202c7b4b40025f3ea4721c52e7f78545e3b07. We shouldn't leave the device as suspended state after module freed, it is not good to do runtime suspend at driver free, here revert this fixing, and replace it with the procedure: suspends firmware ==> frees runtime modules ==> unloads firmware. Signed-off-by: Jie Yang Signed-off-by: Mark Brown --- diff --git a/sound/soc/intel/haswell/sst-haswell-pcm.c b/sound/soc/intel/haswell/sst-haswell-pcm.c index bd96629..23ae040 100644 --- a/sound/soc/intel/haswell/sst-haswell-pcm.c +++ b/sound/soc/intel/haswell/sst-haswell-pcm.c @@ -1118,10 +1118,8 @@ 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)