7 Jan
2016
7 Jan
'16
9:25 a.m.
On Wed, Jan 06, 2016 at 02:08:57PM +0100, Takashi Iwai wrote:
On Tue, 05 Jan 2016 12:44:49 +0100, Vinod Koul wrote:
- pm_runtime_put(platform->dev);
- for (i = 0; i < rtd->num_codecs; i++)
pm_runtime_put(rtd->codec_dais[i]->dev);- pm_runtime_put(cpu_dai->dev);
- pm_runtime_mark_last_busy(platform->dev);
- pm_runtime_put_autosuspend(platform->dev);
- for (i = 0; i < rtd->num_codecs; i++) {
pm_runtime_mark_last_busy(rtd->codec_dais[i]->dev);pm_runtime_put_autosuspend(rtd->codec_dais[i]->dev);- }
- pm_runtime_mark_last_busy(cpu_dai->dev);
- pm_runtime_put_autosuspend(cpu_dai->dev);
- for (i = 0; i < rtd->num_codecs; i++) { if (!rtd->codec_dais[i]->active) pinctrl_pm_select_sleep_state(rtd->codec_dais[i]->dev);
Lots of duplicated codes there... Worth to have a common helper?
I certainly think so :) will do this after merge window
--
~Vinod