xref: /openbmc/linux/sound/soc/sof/intel/hda-dsp.c (revision 2a51c0f81adda8cc32b02e8ca7b7d9d13ad4376a)
1e149ca29SPierre-Louis Bossart // SPDX-License-Identifier: (GPL-2.0-only OR BSD-3-Clause)
2747503b1SLiam Girdwood //
3747503b1SLiam Girdwood // This file is provided under a dual BSD/GPLv2 license.  When using or
4747503b1SLiam Girdwood // redistributing this file, you may do so under either license.
5747503b1SLiam Girdwood //
6747503b1SLiam Girdwood // Copyright(c) 2018 Intel Corporation. All rights reserved.
7747503b1SLiam Girdwood //
8747503b1SLiam Girdwood // Authors: Liam Girdwood <liam.r.girdwood@linux.intel.com>
9747503b1SLiam Girdwood //	    Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
10747503b1SLiam Girdwood //	    Rander Wang <rander.wang@intel.com>
11747503b1SLiam Girdwood //          Keyon Jie <yang.jie@linux.intel.com>
12747503b1SLiam Girdwood //
13747503b1SLiam Girdwood 
14747503b1SLiam Girdwood /*
15747503b1SLiam Girdwood  * Hardware interface for generic Intel audio DSP HDA IP
16747503b1SLiam Girdwood  */
17747503b1SLiam Girdwood 
18851fd873SRanjani Sridharan #include <linux/module.h>
19747503b1SLiam Girdwood #include <sound/hdaudio_ext.h>
20747503b1SLiam Girdwood #include <sound/hda_register.h>
2163e51fd3SRanjani Sridharan #include "../sof-audio.h"
22747503b1SLiam Girdwood #include "../ops.h"
23747503b1SLiam Girdwood #include "hda.h"
24534037fdSKeyon Jie #include "hda-ipc.h"
25747503b1SLiam Girdwood 
26851fd873SRanjani Sridharan static bool hda_enable_trace_D0I3_S0;
27851fd873SRanjani Sridharan #if IS_ENABLED(CONFIG_SND_SOC_SOF_DEBUG)
28851fd873SRanjani Sridharan module_param_named(enable_trace_D0I3_S0, hda_enable_trace_D0I3_S0, bool, 0444);
29851fd873SRanjani Sridharan MODULE_PARM_DESC(enable_trace_D0I3_S0,
30851fd873SRanjani Sridharan 		 "SOF HDA enable trace when the DSP is in D0I3 in S0");
31851fd873SRanjani Sridharan #endif
32851fd873SRanjani Sridharan 
33747503b1SLiam Girdwood /*
34747503b1SLiam Girdwood  * DSP Core control.
35747503b1SLiam Girdwood  */
36747503b1SLiam Girdwood 
37189bf1deSPeter Ujfalusi static int hda_dsp_core_reset_enter(struct snd_sof_dev *sdev, unsigned int core_mask)
38747503b1SLiam Girdwood {
39747503b1SLiam Girdwood 	u32 adspcs;
40747503b1SLiam Girdwood 	u32 reset;
41747503b1SLiam Girdwood 	int ret;
42747503b1SLiam Girdwood 
43747503b1SLiam Girdwood 	/* set reset bits for cores */
44747503b1SLiam Girdwood 	reset = HDA_DSP_ADSPCS_CRST_MASK(core_mask);
45747503b1SLiam Girdwood 	snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
46747503b1SLiam Girdwood 					 HDA_DSP_REG_ADSPCS,
47bed5ed64SJulia Lawall 					 reset, reset);
48747503b1SLiam Girdwood 
49747503b1SLiam Girdwood 	/* poll with timeout to check if operation successful */
50747503b1SLiam Girdwood 	ret = snd_sof_dsp_read_poll_timeout(sdev, HDA_DSP_BAR,
51747503b1SLiam Girdwood 					HDA_DSP_REG_ADSPCS, adspcs,
52747503b1SLiam Girdwood 					((adspcs & reset) == reset),
53747503b1SLiam Girdwood 					HDA_DSP_REG_POLL_INTERVAL_US,
54747503b1SLiam Girdwood 					HDA_DSP_RESET_TIMEOUT_US);
556a414489SPierre-Louis Bossart 	if (ret < 0) {
566a414489SPierre-Louis Bossart 		dev_err(sdev->dev,
576a414489SPierre-Louis Bossart 			"error: %s: timeout on HDA_DSP_REG_ADSPCS read\n",
586a414489SPierre-Louis Bossart 			__func__);
596a414489SPierre-Louis Bossart 		return ret;
606a414489SPierre-Louis Bossart 	}
61747503b1SLiam Girdwood 
62747503b1SLiam Girdwood 	/* has core entered reset ? */
63747503b1SLiam Girdwood 	adspcs = snd_sof_dsp_read(sdev, HDA_DSP_BAR,
64747503b1SLiam Girdwood 				  HDA_DSP_REG_ADSPCS);
65747503b1SLiam Girdwood 	if ((adspcs & HDA_DSP_ADSPCS_CRST_MASK(core_mask)) !=
66747503b1SLiam Girdwood 		HDA_DSP_ADSPCS_CRST_MASK(core_mask)) {
67747503b1SLiam Girdwood 		dev_err(sdev->dev,
68747503b1SLiam Girdwood 			"error: reset enter failed: core_mask %x adspcs 0x%x\n",
69747503b1SLiam Girdwood 			core_mask, adspcs);
70747503b1SLiam Girdwood 		ret = -EIO;
71747503b1SLiam Girdwood 	}
72747503b1SLiam Girdwood 
73747503b1SLiam Girdwood 	return ret;
74747503b1SLiam Girdwood }
75747503b1SLiam Girdwood 
76189bf1deSPeter Ujfalusi static int hda_dsp_core_reset_leave(struct snd_sof_dev *sdev, unsigned int core_mask)
77747503b1SLiam Girdwood {
78747503b1SLiam Girdwood 	unsigned int crst;
79747503b1SLiam Girdwood 	u32 adspcs;
80747503b1SLiam Girdwood 	int ret;
81747503b1SLiam Girdwood 
82747503b1SLiam Girdwood 	/* clear reset bits for cores */
83747503b1SLiam Girdwood 	snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
84747503b1SLiam Girdwood 					 HDA_DSP_REG_ADSPCS,
85747503b1SLiam Girdwood 					 HDA_DSP_ADSPCS_CRST_MASK(core_mask),
86747503b1SLiam Girdwood 					 0);
87747503b1SLiam Girdwood 
88747503b1SLiam Girdwood 	/* poll with timeout to check if operation successful */
89747503b1SLiam Girdwood 	crst = HDA_DSP_ADSPCS_CRST_MASK(core_mask);
90747503b1SLiam Girdwood 	ret = snd_sof_dsp_read_poll_timeout(sdev, HDA_DSP_BAR,
91747503b1SLiam Girdwood 					    HDA_DSP_REG_ADSPCS, adspcs,
92747503b1SLiam Girdwood 					    !(adspcs & crst),
93747503b1SLiam Girdwood 					    HDA_DSP_REG_POLL_INTERVAL_US,
94747503b1SLiam Girdwood 					    HDA_DSP_RESET_TIMEOUT_US);
95747503b1SLiam Girdwood 
966a414489SPierre-Louis Bossart 	if (ret < 0) {
976a414489SPierre-Louis Bossart 		dev_err(sdev->dev,
986a414489SPierre-Louis Bossart 			"error: %s: timeout on HDA_DSP_REG_ADSPCS read\n",
996a414489SPierre-Louis Bossart 			__func__);
1006a414489SPierre-Louis Bossart 		return ret;
1016a414489SPierre-Louis Bossart 	}
1026a414489SPierre-Louis Bossart 
103747503b1SLiam Girdwood 	/* has core left reset ? */
104747503b1SLiam Girdwood 	adspcs = snd_sof_dsp_read(sdev, HDA_DSP_BAR,
105747503b1SLiam Girdwood 				  HDA_DSP_REG_ADSPCS);
106747503b1SLiam Girdwood 	if ((adspcs & HDA_DSP_ADSPCS_CRST_MASK(core_mask)) != 0) {
107747503b1SLiam Girdwood 		dev_err(sdev->dev,
108747503b1SLiam Girdwood 			"error: reset leave failed: core_mask %x adspcs 0x%x\n",
109747503b1SLiam Girdwood 			core_mask, adspcs);
110747503b1SLiam Girdwood 		ret = -EIO;
111747503b1SLiam Girdwood 	}
112747503b1SLiam Girdwood 
113747503b1SLiam Girdwood 	return ret;
114747503b1SLiam Girdwood }
115747503b1SLiam Girdwood 
116189bf1deSPeter Ujfalusi static int hda_dsp_core_stall_reset(struct snd_sof_dev *sdev, unsigned int core_mask)
117747503b1SLiam Girdwood {
118747503b1SLiam Girdwood 	/* stall core */
119747503b1SLiam Girdwood 	snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
120747503b1SLiam Girdwood 					 HDA_DSP_REG_ADSPCS,
121747503b1SLiam Girdwood 					 HDA_DSP_ADSPCS_CSTALL_MASK(core_mask),
122747503b1SLiam Girdwood 					 HDA_DSP_ADSPCS_CSTALL_MASK(core_mask));
123747503b1SLiam Girdwood 
124747503b1SLiam Girdwood 	/* set reset state */
125747503b1SLiam Girdwood 	return hda_dsp_core_reset_enter(sdev, core_mask);
126747503b1SLiam Girdwood }
127747503b1SLiam Girdwood 
128189bf1deSPeter Ujfalusi static bool hda_dsp_core_is_enabled(struct snd_sof_dev *sdev, unsigned int core_mask)
129189bf1deSPeter Ujfalusi {
130189bf1deSPeter Ujfalusi 	int val;
131189bf1deSPeter Ujfalusi 	bool is_enable;
132189bf1deSPeter Ujfalusi 
133189bf1deSPeter Ujfalusi 	val = snd_sof_dsp_read(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPCS);
134189bf1deSPeter Ujfalusi 
135189bf1deSPeter Ujfalusi #define MASK_IS_EQUAL(v, m, field) ({	\
136189bf1deSPeter Ujfalusi 	u32 _m = field(m);		\
137189bf1deSPeter Ujfalusi 	((v) & _m) == _m;		\
138189bf1deSPeter Ujfalusi })
139189bf1deSPeter Ujfalusi 
140189bf1deSPeter Ujfalusi 	is_enable = MASK_IS_EQUAL(val, core_mask, HDA_DSP_ADSPCS_CPA_MASK) &&
141189bf1deSPeter Ujfalusi 		MASK_IS_EQUAL(val, core_mask, HDA_DSP_ADSPCS_SPA_MASK) &&
142189bf1deSPeter Ujfalusi 		!(val & HDA_DSP_ADSPCS_CRST_MASK(core_mask)) &&
143189bf1deSPeter Ujfalusi 		!(val & HDA_DSP_ADSPCS_CSTALL_MASK(core_mask));
144189bf1deSPeter Ujfalusi 
145189bf1deSPeter Ujfalusi #undef MASK_IS_EQUAL
146189bf1deSPeter Ujfalusi 
147189bf1deSPeter Ujfalusi 	dev_dbg(sdev->dev, "DSP core(s) enabled? %d : core_mask %x\n",
148189bf1deSPeter Ujfalusi 		is_enable, core_mask);
149189bf1deSPeter Ujfalusi 
150189bf1deSPeter Ujfalusi 	return is_enable;
151189bf1deSPeter Ujfalusi }
152189bf1deSPeter Ujfalusi 
153747503b1SLiam Girdwood int hda_dsp_core_run(struct snd_sof_dev *sdev, unsigned int core_mask)
154747503b1SLiam Girdwood {
155747503b1SLiam Girdwood 	int ret;
156747503b1SLiam Girdwood 
157747503b1SLiam Girdwood 	/* leave reset state */
158747503b1SLiam Girdwood 	ret = hda_dsp_core_reset_leave(sdev, core_mask);
159747503b1SLiam Girdwood 	if (ret < 0)
160747503b1SLiam Girdwood 		return ret;
161747503b1SLiam Girdwood 
162747503b1SLiam Girdwood 	/* run core */
163747503b1SLiam Girdwood 	dev_dbg(sdev->dev, "unstall/run core: core_mask = %x\n", core_mask);
164747503b1SLiam Girdwood 	snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
165747503b1SLiam Girdwood 					 HDA_DSP_REG_ADSPCS,
166747503b1SLiam Girdwood 					 HDA_DSP_ADSPCS_CSTALL_MASK(core_mask),
167747503b1SLiam Girdwood 					 0);
168747503b1SLiam Girdwood 
169747503b1SLiam Girdwood 	/* is core now running ? */
170747503b1SLiam Girdwood 	if (!hda_dsp_core_is_enabled(sdev, core_mask)) {
171747503b1SLiam Girdwood 		hda_dsp_core_stall_reset(sdev, core_mask);
172747503b1SLiam Girdwood 		dev_err(sdev->dev, "error: DSP start core failed: core_mask %x\n",
173747503b1SLiam Girdwood 			core_mask);
174747503b1SLiam Girdwood 		ret = -EIO;
175747503b1SLiam Girdwood 	}
176747503b1SLiam Girdwood 
177747503b1SLiam Girdwood 	return ret;
178747503b1SLiam Girdwood }
179747503b1SLiam Girdwood 
180747503b1SLiam Girdwood /*
181747503b1SLiam Girdwood  * Power Management.
182747503b1SLiam Girdwood  */
183747503b1SLiam Girdwood 
184189bf1deSPeter Ujfalusi static int hda_dsp_core_power_up(struct snd_sof_dev *sdev, unsigned int core_mask)
185747503b1SLiam Girdwood {
186747503b1SLiam Girdwood 	unsigned int cpa;
187747503b1SLiam Girdwood 	u32 adspcs;
188747503b1SLiam Girdwood 	int ret;
189747503b1SLiam Girdwood 
190747503b1SLiam Girdwood 	/* update bits */
191747503b1SLiam Girdwood 	snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPCS,
192747503b1SLiam Girdwood 				HDA_DSP_ADSPCS_SPA_MASK(core_mask),
193747503b1SLiam Girdwood 				HDA_DSP_ADSPCS_SPA_MASK(core_mask));
194747503b1SLiam Girdwood 
195747503b1SLiam Girdwood 	/* poll with timeout to check if operation successful */
196747503b1SLiam Girdwood 	cpa = HDA_DSP_ADSPCS_CPA_MASK(core_mask);
197747503b1SLiam Girdwood 	ret = snd_sof_dsp_read_poll_timeout(sdev, HDA_DSP_BAR,
198747503b1SLiam Girdwood 					    HDA_DSP_REG_ADSPCS, adspcs,
199747503b1SLiam Girdwood 					    (adspcs & cpa) == cpa,
200747503b1SLiam Girdwood 					    HDA_DSP_REG_POLL_INTERVAL_US,
201747503b1SLiam Girdwood 					    HDA_DSP_RESET_TIMEOUT_US);
2026a414489SPierre-Louis Bossart 	if (ret < 0) {
2036a414489SPierre-Louis Bossart 		dev_err(sdev->dev,
2046a414489SPierre-Louis Bossart 			"error: %s: timeout on HDA_DSP_REG_ADSPCS read\n",
2056a414489SPierre-Louis Bossart 			__func__);
2066a414489SPierre-Louis Bossart 		return ret;
2076a414489SPierre-Louis Bossart 	}
208747503b1SLiam Girdwood 
209747503b1SLiam Girdwood 	/* did core power up ? */
210747503b1SLiam Girdwood 	adspcs = snd_sof_dsp_read(sdev, HDA_DSP_BAR,
211747503b1SLiam Girdwood 				  HDA_DSP_REG_ADSPCS);
212747503b1SLiam Girdwood 	if ((adspcs & HDA_DSP_ADSPCS_CPA_MASK(core_mask)) !=
213747503b1SLiam Girdwood 		HDA_DSP_ADSPCS_CPA_MASK(core_mask)) {
214747503b1SLiam Girdwood 		dev_err(sdev->dev,
215747503b1SLiam Girdwood 			"error: power up core failed core_mask %xadspcs 0x%x\n",
216747503b1SLiam Girdwood 			core_mask, adspcs);
217747503b1SLiam Girdwood 		ret = -EIO;
218747503b1SLiam Girdwood 	}
219747503b1SLiam Girdwood 
220747503b1SLiam Girdwood 	return ret;
221747503b1SLiam Girdwood }
222747503b1SLiam Girdwood 
223189bf1deSPeter Ujfalusi static int hda_dsp_core_power_down(struct snd_sof_dev *sdev, unsigned int core_mask)
224747503b1SLiam Girdwood {
225747503b1SLiam Girdwood 	u32 adspcs;
2266a414489SPierre-Louis Bossart 	int ret;
227747503b1SLiam Girdwood 
228747503b1SLiam Girdwood 	/* update bits */
229747503b1SLiam Girdwood 	snd_sof_dsp_update_bits_unlocked(sdev, HDA_DSP_BAR,
230747503b1SLiam Girdwood 					 HDA_DSP_REG_ADSPCS,
231747503b1SLiam Girdwood 					 HDA_DSP_ADSPCS_SPA_MASK(core_mask), 0);
232747503b1SLiam Girdwood 
2336a414489SPierre-Louis Bossart 	ret = snd_sof_dsp_read_poll_timeout(sdev, HDA_DSP_BAR,
234747503b1SLiam Girdwood 				HDA_DSP_REG_ADSPCS, adspcs,
235fd829918SPan Xiuli 				!(adspcs & HDA_DSP_ADSPCS_CPA_MASK(core_mask)),
236747503b1SLiam Girdwood 				HDA_DSP_REG_POLL_INTERVAL_US,
237747503b1SLiam Girdwood 				HDA_DSP_PD_TIMEOUT * USEC_PER_MSEC);
2386a414489SPierre-Louis Bossart 	if (ret < 0)
2396a414489SPierre-Louis Bossart 		dev_err(sdev->dev,
2406a414489SPierre-Louis Bossart 			"error: %s: timeout on HDA_DSP_REG_ADSPCS read\n",
2416a414489SPierre-Louis Bossart 			__func__);
2426a414489SPierre-Louis Bossart 
2436a414489SPierre-Louis Bossart 	return ret;
244747503b1SLiam Girdwood }
245747503b1SLiam Girdwood 
246747503b1SLiam Girdwood int hda_dsp_enable_core(struct snd_sof_dev *sdev, unsigned int core_mask)
247747503b1SLiam Girdwood {
248914fab3bSRanjani Sridharan 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
249914fab3bSRanjani Sridharan 	const struct sof_intel_dsp_desc *chip = hda->desc;
250747503b1SLiam Girdwood 	int ret;
251747503b1SLiam Girdwood 
252914fab3bSRanjani Sridharan 	/* restrict core_mask to host managed cores mask */
253914fab3bSRanjani Sridharan 	core_mask &= chip->host_managed_cores_mask;
254914fab3bSRanjani Sridharan 
255914fab3bSRanjani Sridharan 	/* return if core_mask is not valid or cores are already enabled */
256914fab3bSRanjani Sridharan 	if (!core_mask || hda_dsp_core_is_enabled(sdev, core_mask))
257747503b1SLiam Girdwood 		return 0;
258747503b1SLiam Girdwood 
259747503b1SLiam Girdwood 	/* power up */
260747503b1SLiam Girdwood 	ret = hda_dsp_core_power_up(sdev, core_mask);
261747503b1SLiam Girdwood 	if (ret < 0) {
262747503b1SLiam Girdwood 		dev_err(sdev->dev, "error: dsp core power up failed: core_mask %x\n",
263747503b1SLiam Girdwood 			core_mask);
264747503b1SLiam Girdwood 		return ret;
265747503b1SLiam Girdwood 	}
266747503b1SLiam Girdwood 
267747503b1SLiam Girdwood 	return hda_dsp_core_run(sdev, core_mask);
268747503b1SLiam Girdwood }
269747503b1SLiam Girdwood 
270747503b1SLiam Girdwood int hda_dsp_core_reset_power_down(struct snd_sof_dev *sdev,
271747503b1SLiam Girdwood 				  unsigned int core_mask)
272747503b1SLiam Girdwood {
273914fab3bSRanjani Sridharan 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
274914fab3bSRanjani Sridharan 	const struct sof_intel_dsp_desc *chip = hda->desc;
275747503b1SLiam Girdwood 	int ret;
276747503b1SLiam Girdwood 
277914fab3bSRanjani Sridharan 	/* restrict core_mask to host managed cores mask */
278914fab3bSRanjani Sridharan 	core_mask &= chip->host_managed_cores_mask;
279914fab3bSRanjani Sridharan 
280914fab3bSRanjani Sridharan 	/* return if core_mask is not valid */
281914fab3bSRanjani Sridharan 	if (!core_mask)
282914fab3bSRanjani Sridharan 		return 0;
283914fab3bSRanjani Sridharan 
284747503b1SLiam Girdwood 	/* place core in reset prior to power down */
285747503b1SLiam Girdwood 	ret = hda_dsp_core_stall_reset(sdev, core_mask);
286747503b1SLiam Girdwood 	if (ret < 0) {
287747503b1SLiam Girdwood 		dev_err(sdev->dev, "error: dsp core reset failed: core_mask %x\n",
288747503b1SLiam Girdwood 			core_mask);
289747503b1SLiam Girdwood 		return ret;
290747503b1SLiam Girdwood 	}
291747503b1SLiam Girdwood 
292747503b1SLiam Girdwood 	/* power down core */
293747503b1SLiam Girdwood 	ret = hda_dsp_core_power_down(sdev, core_mask);
294747503b1SLiam Girdwood 	if (ret < 0) {
295747503b1SLiam Girdwood 		dev_err(sdev->dev, "error: dsp core power down fail mask %x: %d\n",
296747503b1SLiam Girdwood 			core_mask, ret);
297747503b1SLiam Girdwood 		return ret;
298747503b1SLiam Girdwood 	}
299747503b1SLiam Girdwood 
300747503b1SLiam Girdwood 	/* make sure we are in OFF state */
301747503b1SLiam Girdwood 	if (hda_dsp_core_is_enabled(sdev, core_mask)) {
302747503b1SLiam Girdwood 		dev_err(sdev->dev, "error: dsp core disable fail mask %x: %d\n",
303747503b1SLiam Girdwood 			core_mask, ret);
304747503b1SLiam Girdwood 		ret = -EIO;
305747503b1SLiam Girdwood 	}
306747503b1SLiam Girdwood 
307747503b1SLiam Girdwood 	return ret;
308747503b1SLiam Girdwood }
309747503b1SLiam Girdwood 
310747503b1SLiam Girdwood void hda_dsp_ipc_int_enable(struct snd_sof_dev *sdev)
311747503b1SLiam Girdwood {
312747503b1SLiam Girdwood 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
313747503b1SLiam Girdwood 	const struct sof_intel_dsp_desc *chip = hda->desc;
314747503b1SLiam Girdwood 
315747503b1SLiam Girdwood 	/* enable IPC DONE and BUSY interrupts */
316747503b1SLiam Girdwood 	snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR, chip->ipc_ctl,
317747503b1SLiam Girdwood 			HDA_DSP_REG_HIPCCTL_DONE | HDA_DSP_REG_HIPCCTL_BUSY,
318747503b1SLiam Girdwood 			HDA_DSP_REG_HIPCCTL_DONE | HDA_DSP_REG_HIPCCTL_BUSY);
319747503b1SLiam Girdwood 
320747503b1SLiam Girdwood 	/* enable IPC interrupt */
321747503b1SLiam Girdwood 	snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPIC,
322747503b1SLiam Girdwood 				HDA_DSP_ADSPIC_IPC, HDA_DSP_ADSPIC_IPC);
323747503b1SLiam Girdwood }
324747503b1SLiam Girdwood 
325747503b1SLiam Girdwood void hda_dsp_ipc_int_disable(struct snd_sof_dev *sdev)
326747503b1SLiam Girdwood {
327747503b1SLiam Girdwood 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
328747503b1SLiam Girdwood 	const struct sof_intel_dsp_desc *chip = hda->desc;
329747503b1SLiam Girdwood 
330747503b1SLiam Girdwood 	/* disable IPC interrupt */
331747503b1SLiam Girdwood 	snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR, HDA_DSP_REG_ADSPIC,
332747503b1SLiam Girdwood 				HDA_DSP_ADSPIC_IPC, 0);
333747503b1SLiam Girdwood 
334747503b1SLiam Girdwood 	/* disable IPC BUSY and DONE interrupt */
335747503b1SLiam Girdwood 	snd_sof_dsp_update_bits(sdev, HDA_DSP_BAR, chip->ipc_ctl,
336747503b1SLiam Girdwood 			HDA_DSP_REG_HIPCCTL_BUSY | HDA_DSP_REG_HIPCCTL_DONE, 0);
337747503b1SLiam Girdwood }
338747503b1SLiam Girdwood 
33965c56f5dSRanjani Sridharan static int hda_dsp_wait_d0i3c_done(struct snd_sof_dev *sdev)
34062f8f766SKeyon Jie {
34162f8f766SKeyon Jie 	struct hdac_bus *bus = sof_to_bus(sdev);
34265c56f5dSRanjani Sridharan 	int retry = HDA_DSP_REG_POLL_RETRY_COUNT;
34362f8f766SKeyon Jie 
34462f8f766SKeyon Jie 	while (snd_hdac_chip_readb(bus, VS_D0I3C) & SOF_HDA_VS_D0I3C_CIP) {
34562f8f766SKeyon Jie 		if (!retry--)
34662f8f766SKeyon Jie 			return -ETIMEDOUT;
34762f8f766SKeyon Jie 		usleep_range(10, 15);
34862f8f766SKeyon Jie 	}
34962f8f766SKeyon Jie 
35062f8f766SKeyon Jie 	return 0;
35162f8f766SKeyon Jie }
35262f8f766SKeyon Jie 
353534037fdSKeyon Jie static int hda_dsp_send_pm_gate_ipc(struct snd_sof_dev *sdev, u32 flags)
354534037fdSKeyon Jie {
355534037fdSKeyon Jie 	struct sof_ipc_pm_gate pm_gate;
356534037fdSKeyon Jie 	struct sof_ipc_reply reply;
357534037fdSKeyon Jie 
358534037fdSKeyon Jie 	memset(&pm_gate, 0, sizeof(pm_gate));
359534037fdSKeyon Jie 
360534037fdSKeyon Jie 	/* configure pm_gate ipc message */
361534037fdSKeyon Jie 	pm_gate.hdr.size = sizeof(pm_gate);
362534037fdSKeyon Jie 	pm_gate.hdr.cmd = SOF_IPC_GLB_PM_MSG | SOF_IPC_PM_GATE;
363534037fdSKeyon Jie 	pm_gate.flags = flags;
364534037fdSKeyon Jie 
365534037fdSKeyon Jie 	/* send pm_gate ipc to dsp */
366*2a51c0f8SPeter Ujfalusi 	return sof_ipc_tx_message_no_pm(sdev->ipc, &pm_gate, sizeof(pm_gate),
367*2a51c0f8SPeter Ujfalusi 					&reply, sizeof(reply));
368534037fdSKeyon Jie }
369534037fdSKeyon Jie 
37061e285caSRanjani Sridharan static int hda_dsp_update_d0i3c_register(struct snd_sof_dev *sdev, u8 value)
37162f8f766SKeyon Jie {
37262f8f766SKeyon Jie 	struct hdac_bus *bus = sof_to_bus(sdev);
37362f8f766SKeyon Jie 	int ret;
37462f8f766SKeyon Jie 
37562f8f766SKeyon Jie 	/* Write to D0I3C after Command-In-Progress bit is cleared */
37665c56f5dSRanjani Sridharan 	ret = hda_dsp_wait_d0i3c_done(sdev);
37762f8f766SKeyon Jie 	if (ret < 0) {
378aae7c82dSKeyon Jie 		dev_err(bus->dev, "CIP timeout before D0I3C update!\n");
37962f8f766SKeyon Jie 		return ret;
38062f8f766SKeyon Jie 	}
38162f8f766SKeyon Jie 
38262f8f766SKeyon Jie 	/* Update D0I3C register */
38362f8f766SKeyon Jie 	snd_hdac_chip_updateb(bus, VS_D0I3C, SOF_HDA_VS_D0I3C_I3, value);
38462f8f766SKeyon Jie 
38562f8f766SKeyon Jie 	/* Wait for cmd in progress to be cleared before exiting the function */
38665c56f5dSRanjani Sridharan 	ret = hda_dsp_wait_d0i3c_done(sdev);
38762f8f766SKeyon Jie 	if (ret < 0) {
388aae7c82dSKeyon Jie 		dev_err(bus->dev, "CIP timeout after D0I3C update!\n");
38962f8f766SKeyon Jie 		return ret;
39062f8f766SKeyon Jie 	}
39162f8f766SKeyon Jie 
39262f8f766SKeyon Jie 	dev_vdbg(bus->dev, "D0I3C updated, register = 0x%x\n",
39362f8f766SKeyon Jie 		 snd_hdac_chip_readb(bus, VS_D0I3C));
39462f8f766SKeyon Jie 
39561e285caSRanjani Sridharan 	return 0;
39661e285caSRanjani Sridharan }
397534037fdSKeyon Jie 
39861e285caSRanjani Sridharan static int hda_dsp_set_D0_state(struct snd_sof_dev *sdev,
39961e285caSRanjani Sridharan 				const struct sof_dsp_power_state *target_state)
40061e285caSRanjani Sridharan {
40161e285caSRanjani Sridharan 	u32 flags = 0;
40261e285caSRanjani Sridharan 	int ret;
40361e285caSRanjani Sridharan 	u8 value = 0;
40461e285caSRanjani Sridharan 
40561e285caSRanjani Sridharan 	/*
40661e285caSRanjani Sridharan 	 * Sanity check for illegal state transitions
40761e285caSRanjani Sridharan 	 * The only allowed transitions are:
40861e285caSRanjani Sridharan 	 * 1. D3 -> D0I0
40961e285caSRanjani Sridharan 	 * 2. D0I0 -> D0I3
41061e285caSRanjani Sridharan 	 * 3. D0I3 -> D0I0
41161e285caSRanjani Sridharan 	 */
41261e285caSRanjani Sridharan 	switch (sdev->dsp_power_state.state) {
41361e285caSRanjani Sridharan 	case SOF_DSP_PM_D0:
41461e285caSRanjani Sridharan 		/* Follow the sequence below for D0 substate transitions */
41561e285caSRanjani Sridharan 		break;
41661e285caSRanjani Sridharan 	case SOF_DSP_PM_D3:
41761e285caSRanjani Sridharan 		/* Follow regular flow for D3 -> D0 transition */
41861e285caSRanjani Sridharan 		return 0;
41961e285caSRanjani Sridharan 	default:
42061e285caSRanjani Sridharan 		dev_err(sdev->dev, "error: transition from %d to %d not allowed\n",
42161e285caSRanjani Sridharan 			sdev->dsp_power_state.state, target_state->state);
42261e285caSRanjani Sridharan 		return -EINVAL;
42361e285caSRanjani Sridharan 	}
42461e285caSRanjani Sridharan 
42561e285caSRanjani Sridharan 	/* Set flags and register value for D0 target substate */
42661e285caSRanjani Sridharan 	if (target_state->substate == SOF_HDA_DSP_PM_D0I3) {
42761e285caSRanjani Sridharan 		value = SOF_HDA_VS_D0I3C_I3;
42861e285caSRanjani Sridharan 
429851fd873SRanjani Sridharan 		/*
43079560b8aSMarcin Rajwa 		 * Trace DMA need to be disabled when the DSP enters
43179560b8aSMarcin Rajwa 		 * D0I3 for S0Ix suspend, but it can be kept enabled
43279560b8aSMarcin Rajwa 		 * when the DSP enters D0I3 while the system is in S0
43379560b8aSMarcin Rajwa 		 * for debug purpose.
434851fd873SRanjani Sridharan 		 */
43579560b8aSMarcin Rajwa 		if (!sdev->dtrace_is_supported ||
43679560b8aSMarcin Rajwa 		    !hda_enable_trace_D0I3_S0 ||
437851fd873SRanjani Sridharan 		    sdev->system_suspend_target != SOF_SUSPEND_NONE)
43861e285caSRanjani Sridharan 			flags = HDA_PM_NO_DMA_TRACE;
43961e285caSRanjani Sridharan 	} else {
44061e285caSRanjani Sridharan 		/* prevent power gating in D0I0 */
44161e285caSRanjani Sridharan 		flags = HDA_PM_PPG;
44261e285caSRanjani Sridharan 	}
44361e285caSRanjani Sridharan 
44461e285caSRanjani Sridharan 	/* update D0I3C register */
44561e285caSRanjani Sridharan 	ret = hda_dsp_update_d0i3c_register(sdev, value);
446534037fdSKeyon Jie 	if (ret < 0)
44761e285caSRanjani Sridharan 		return ret;
44861e285caSRanjani Sridharan 
44961e285caSRanjani Sridharan 	/*
45061e285caSRanjani Sridharan 	 * Notify the DSP of the state change.
45161e285caSRanjani Sridharan 	 * If this IPC fails, revert the D0I3C register update in order
45261e285caSRanjani Sridharan 	 * to prevent partial state change.
45361e285caSRanjani Sridharan 	 */
45461e285caSRanjani Sridharan 	ret = hda_dsp_send_pm_gate_ipc(sdev, flags);
45561e285caSRanjani Sridharan 	if (ret < 0) {
456534037fdSKeyon Jie 		dev_err(sdev->dev,
457534037fdSKeyon Jie 			"error: PM_GATE ipc error %d\n", ret);
45861e285caSRanjani Sridharan 		goto revert;
45961e285caSRanjani Sridharan 	}
46061e285caSRanjani Sridharan 
46161e285caSRanjani Sridharan 	return ret;
46261e285caSRanjani Sridharan 
46361e285caSRanjani Sridharan revert:
46461e285caSRanjani Sridharan 	/* fallback to the previous register value */
46561e285caSRanjani Sridharan 	value = value ? 0 : SOF_HDA_VS_D0I3C_I3;
46661e285caSRanjani Sridharan 
46761e285caSRanjani Sridharan 	/*
46861e285caSRanjani Sridharan 	 * This can fail but return the IPC error to signal that
46961e285caSRanjani Sridharan 	 * the state change failed.
47061e285caSRanjani Sridharan 	 */
47161e285caSRanjani Sridharan 	hda_dsp_update_d0i3c_register(sdev, value);
472534037fdSKeyon Jie 
473534037fdSKeyon Jie 	return ret;
47462f8f766SKeyon Jie }
47562f8f766SKeyon Jie 
47666de6bebSRanjani Sridharan /* helper to log DSP state */
47766de6bebSRanjani Sridharan static void hda_dsp_state_log(struct snd_sof_dev *sdev)
47866de6bebSRanjani Sridharan {
47966de6bebSRanjani Sridharan 	switch (sdev->dsp_power_state.state) {
48066de6bebSRanjani Sridharan 	case SOF_DSP_PM_D0:
48166de6bebSRanjani Sridharan 		switch (sdev->dsp_power_state.substate) {
48266de6bebSRanjani Sridharan 		case SOF_HDA_DSP_PM_D0I0:
48366de6bebSRanjani Sridharan 			dev_dbg(sdev->dev, "Current DSP power state: D0I0\n");
48466de6bebSRanjani Sridharan 			break;
48566de6bebSRanjani Sridharan 		case SOF_HDA_DSP_PM_D0I3:
48666de6bebSRanjani Sridharan 			dev_dbg(sdev->dev, "Current DSP power state: D0I3\n");
48766de6bebSRanjani Sridharan 			break;
48866de6bebSRanjani Sridharan 		default:
48966de6bebSRanjani Sridharan 			dev_dbg(sdev->dev, "Unknown DSP D0 substate: %d\n",
49066de6bebSRanjani Sridharan 				sdev->dsp_power_state.substate);
49166de6bebSRanjani Sridharan 			break;
49266de6bebSRanjani Sridharan 		}
49366de6bebSRanjani Sridharan 		break;
49466de6bebSRanjani Sridharan 	case SOF_DSP_PM_D1:
49566de6bebSRanjani Sridharan 		dev_dbg(sdev->dev, "Current DSP power state: D1\n");
49666de6bebSRanjani Sridharan 		break;
49766de6bebSRanjani Sridharan 	case SOF_DSP_PM_D2:
49866de6bebSRanjani Sridharan 		dev_dbg(sdev->dev, "Current DSP power state: D2\n");
49966de6bebSRanjani Sridharan 		break;
50066de6bebSRanjani Sridharan 	case SOF_DSP_PM_D3:
50166de6bebSRanjani Sridharan 		dev_dbg(sdev->dev, "Current DSP power state: D3\n");
50266de6bebSRanjani Sridharan 		break;
50366de6bebSRanjani Sridharan 	default:
50466de6bebSRanjani Sridharan 		dev_dbg(sdev->dev, "Unknown DSP power state: %d\n",
50566de6bebSRanjani Sridharan 			sdev->dsp_power_state.state);
50666de6bebSRanjani Sridharan 		break;
50766de6bebSRanjani Sridharan 	}
50866de6bebSRanjani Sridharan }
50966de6bebSRanjani Sridharan 
51061e285caSRanjani Sridharan /*
51161e285caSRanjani Sridharan  * All DSP power state transitions are initiated by the driver.
51261e285caSRanjani Sridharan  * If the requested state change fails, the error is simply returned.
51361e285caSRanjani Sridharan  * Further state transitions are attempted only when the set_power_save() op
51461e285caSRanjani Sridharan  * is called again either because of a new IPC sent to the DSP or
51561e285caSRanjani Sridharan  * during system suspend/resume.
51661e285caSRanjani Sridharan  */
51761e285caSRanjani Sridharan int hda_dsp_set_power_state(struct snd_sof_dev *sdev,
51861e285caSRanjani Sridharan 			    const struct sof_dsp_power_state *target_state)
51961e285caSRanjani Sridharan {
52061e285caSRanjani Sridharan 	int ret = 0;
52161e285caSRanjani Sridharan 
522851fd873SRanjani Sridharan 	/*
523851fd873SRanjani Sridharan 	 * When the DSP is already in D0I3 and the target state is D0I3,
524851fd873SRanjani Sridharan 	 * it could be the case that the DSP is in D0I3 during S0
525851fd873SRanjani Sridharan 	 * and the system is suspending to S0Ix. Therefore,
526851fd873SRanjani Sridharan 	 * hda_dsp_set_D0_state() must be called to disable trace DMA
527851fd873SRanjani Sridharan 	 * by sending the PM_GATE IPC to the FW.
528851fd873SRanjani Sridharan 	 */
529851fd873SRanjani Sridharan 	if (target_state->substate == SOF_HDA_DSP_PM_D0I3 &&
530851fd873SRanjani Sridharan 	    sdev->system_suspend_target == SOF_SUSPEND_S0IX)
531851fd873SRanjani Sridharan 		goto set_state;
532851fd873SRanjani Sridharan 
533851fd873SRanjani Sridharan 	/*
534851fd873SRanjani Sridharan 	 * For all other cases, return without doing anything if
535851fd873SRanjani Sridharan 	 * the DSP is already in the target state.
536851fd873SRanjani Sridharan 	 */
53761e285caSRanjani Sridharan 	if (target_state->state == sdev->dsp_power_state.state &&
53861e285caSRanjani Sridharan 	    target_state->substate == sdev->dsp_power_state.substate)
53961e285caSRanjani Sridharan 		return 0;
54061e285caSRanjani Sridharan 
541851fd873SRanjani Sridharan set_state:
54261e285caSRanjani Sridharan 	switch (target_state->state) {
54361e285caSRanjani Sridharan 	case SOF_DSP_PM_D0:
54461e285caSRanjani Sridharan 		ret = hda_dsp_set_D0_state(sdev, target_state);
54561e285caSRanjani Sridharan 		break;
54661e285caSRanjani Sridharan 	case SOF_DSP_PM_D3:
54761e285caSRanjani Sridharan 		/* The only allowed transition is: D0I0 -> D3 */
54861e285caSRanjani Sridharan 		if (sdev->dsp_power_state.state == SOF_DSP_PM_D0 &&
54961e285caSRanjani Sridharan 		    sdev->dsp_power_state.substate == SOF_HDA_DSP_PM_D0I0)
55061e285caSRanjani Sridharan 			break;
55161e285caSRanjani Sridharan 
55261e285caSRanjani Sridharan 		dev_err(sdev->dev,
55361e285caSRanjani Sridharan 			"error: transition from %d to %d not allowed\n",
55461e285caSRanjani Sridharan 			sdev->dsp_power_state.state, target_state->state);
55561e285caSRanjani Sridharan 		return -EINVAL;
55661e285caSRanjani Sridharan 	default:
55761e285caSRanjani Sridharan 		dev_err(sdev->dev, "error: target state unsupported %d\n",
55861e285caSRanjani Sridharan 			target_state->state);
55961e285caSRanjani Sridharan 		return -EINVAL;
56061e285caSRanjani Sridharan 	}
56161e285caSRanjani Sridharan 	if (ret < 0) {
56261e285caSRanjani Sridharan 		dev_err(sdev->dev,
56361e285caSRanjani Sridharan 			"failed to set requested target DSP state %d substate %d\n",
56461e285caSRanjani Sridharan 			target_state->state, target_state->substate);
56561e285caSRanjani Sridharan 		return ret;
56661e285caSRanjani Sridharan 	}
56761e285caSRanjani Sridharan 
56861e285caSRanjani Sridharan 	sdev->dsp_power_state = *target_state;
56966de6bebSRanjani Sridharan 	hda_dsp_state_log(sdev);
57061e285caSRanjani Sridharan 	return ret;
57161e285caSRanjani Sridharan }
57261e285caSRanjani Sridharan 
57361e285caSRanjani Sridharan /*
57461e285caSRanjani Sridharan  * Audio DSP states may transform as below:-
57561e285caSRanjani Sridharan  *
576207bf12fSRanjani Sridharan  *                                         Opportunistic D0I3 in S0
577207bf12fSRanjani Sridharan  *     Runtime    +---------------------+  Delayed D0i3 work timeout
57861e285caSRanjani Sridharan  *     suspend    |                     +--------------------+
579207bf12fSRanjani Sridharan  *   +------------+       D0I0(active)  |                    |
58061e285caSRanjani Sridharan  *   |            |                     <---------------+    |
581207bf12fSRanjani Sridharan  *   |   +-------->                     |    New IPC	|    |
582207bf12fSRanjani Sridharan  *   |   |Runtime +--^--+---------^--+--+ (via mailbox)	|    |
583207bf12fSRanjani Sridharan  *   |   |resume     |  |         |  |			|    |
584207bf12fSRanjani Sridharan  *   |   |           |  |         |  |			|    |
585207bf12fSRanjani Sridharan  *   |   |     System|  |         |  |			|    |
586207bf12fSRanjani Sridharan  *   |   |     resume|  | S3/S0IX |  |                  |    |
587207bf12fSRanjani Sridharan  *   |   |	     |  | suspend |  | S0IX             |    |
58861e285caSRanjani Sridharan  *   |   |           |  |         |  |suspend           |    |
58961e285caSRanjani Sridharan  *   |   |           |  |         |  |                  |    |
59061e285caSRanjani Sridharan  *   |   |           |  |         |  |                  |    |
59161e285caSRanjani Sridharan  * +-v---+-----------+--v-------+ |  |           +------+----v----+
59261e285caSRanjani Sridharan  * |                            | |  +----------->                |
593207bf12fSRanjani Sridharan  * |       D3 (suspended)       | |              |      D0I3      |
594207bf12fSRanjani Sridharan  * |                            | +--------------+                |
595207bf12fSRanjani Sridharan  * |                            |  System resume |                |
596207bf12fSRanjani Sridharan  * +----------------------------+		 +----------------+
59761e285caSRanjani Sridharan  *
598207bf12fSRanjani Sridharan  * S0IX suspend: The DSP is in D0I3 if any D0I3-compatible streams
599207bf12fSRanjani Sridharan  *		 ignored the suspend trigger. Otherwise the DSP
600207bf12fSRanjani Sridharan  *		 is in D3.
60161e285caSRanjani Sridharan  */
60261e285caSRanjani Sridharan 
6031c38c922SFred Oh static int hda_suspend(struct snd_sof_dev *sdev, bool runtime_suspend)
604747503b1SLiam Girdwood {
605747503b1SLiam Girdwood 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
606747503b1SLiam Girdwood 	const struct sof_intel_dsp_desc *chip = hda->desc;
607747503b1SLiam Girdwood #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
608747503b1SLiam Girdwood 	struct hdac_bus *bus = sof_to_bus(sdev);
609747503b1SLiam Girdwood #endif
610d4165199SRanjani Sridharan 	int ret, j;
611747503b1SLiam Girdwood 
6123eadff56SPierre-Louis Bossart 	hda_sdw_int_enable(sdev, false);
6133eadff56SPierre-Louis Bossart 
614747503b1SLiam Girdwood 	/* disable IPC interrupts */
615747503b1SLiam Girdwood 	hda_dsp_ipc_int_disable(sdev);
616747503b1SLiam Girdwood 
617747503b1SLiam Girdwood #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
618fd572393SKai Vehmanen 	hda_codec_jack_wake_enable(sdev, runtime_suspend);
619fd15f2f5SRander Wang 
620747503b1SLiam Girdwood 	/* power down all hda link */
621747503b1SLiam Girdwood 	snd_hdac_ext_bus_link_power_down_all(bus);
622747503b1SLiam Girdwood #endif
623747503b1SLiam Girdwood 
624747503b1SLiam Girdwood 	/* power down DSP */
625d4165199SRanjani Sridharan 	ret = hda_dsp_core_reset_power_down(sdev, chip->host_managed_cores_mask);
626747503b1SLiam Girdwood 	if (ret < 0) {
627747503b1SLiam Girdwood 		dev_err(sdev->dev,
628747503b1SLiam Girdwood 			"error: failed to power down core during suspend\n");
629747503b1SLiam Girdwood 		return ret;
630747503b1SLiam Girdwood 	}
631747503b1SLiam Girdwood 
632d4165199SRanjani Sridharan 	/* reset ref counts for all cores */
633d4165199SRanjani Sridharan 	for (j = 0; j < chip->cores_num; j++)
634d4165199SRanjani Sridharan 		sdev->dsp_core_ref_count[j] = 0;
635d4165199SRanjani Sridharan 
636747503b1SLiam Girdwood 	/* disable ppcap interrupt */
637747503b1SLiam Girdwood 	hda_dsp_ctrl_ppcap_enable(sdev, false);
638747503b1SLiam Girdwood 	hda_dsp_ctrl_ppcap_int_enable(sdev, false);
639747503b1SLiam Girdwood 
6409a50ee58SZhu Yingjiang 	/* disable hda bus irq and streams */
6419a50ee58SZhu Yingjiang 	hda_dsp_ctrl_stop_chip(sdev);
642747503b1SLiam Girdwood 
643747503b1SLiam Girdwood 	/* disable LP retention mode */
644747503b1SLiam Girdwood 	snd_sof_pci_update_bits(sdev, PCI_PGCTL,
645747503b1SLiam Girdwood 				PCI_PGCTL_LSRMD_MASK, PCI_PGCTL_LSRMD_MASK);
646747503b1SLiam Girdwood 
647747503b1SLiam Girdwood 	/* reset controller */
648747503b1SLiam Girdwood 	ret = hda_dsp_ctrl_link_reset(sdev, true);
649747503b1SLiam Girdwood 	if (ret < 0) {
650747503b1SLiam Girdwood 		dev_err(sdev->dev,
651747503b1SLiam Girdwood 			"error: failed to reset controller during suspend\n");
652747503b1SLiam Girdwood 		return ret;
653747503b1SLiam Girdwood 	}
654747503b1SLiam Girdwood 
655816938b2SKai Vehmanen 	/* display codec can powered off after link reset */
656816938b2SKai Vehmanen 	hda_codec_i915_display_power(sdev, false);
657816938b2SKai Vehmanen 
658747503b1SLiam Girdwood 	return 0;
659747503b1SLiam Girdwood }
660747503b1SLiam Girdwood 
661fd15f2f5SRander Wang static int hda_resume(struct snd_sof_dev *sdev, bool runtime_resume)
662747503b1SLiam Girdwood {
663747503b1SLiam Girdwood #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
664747503b1SLiam Girdwood 	struct hdac_bus *bus = sof_to_bus(sdev);
665747503b1SLiam Girdwood 	struct hdac_ext_link *hlink = NULL;
666747503b1SLiam Girdwood #endif
667747503b1SLiam Girdwood 	int ret;
668747503b1SLiam Girdwood 
669816938b2SKai Vehmanen 	/* display codec must be powered before link reset */
670816938b2SKai Vehmanen 	hda_codec_i915_display_power(sdev, true);
671816938b2SKai Vehmanen 
672747503b1SLiam Girdwood 	/*
673747503b1SLiam Girdwood 	 * clear TCSEL to clear playback on some HD Audio
674747503b1SLiam Girdwood 	 * codecs. PCI TCSEL is defined in the Intel manuals.
675747503b1SLiam Girdwood 	 */
676747503b1SLiam Girdwood 	snd_sof_pci_update_bits(sdev, PCI_TCSEL, 0x07, 0);
677747503b1SLiam Girdwood 
678747503b1SLiam Girdwood 	/* reset and start hda controller */
679747503b1SLiam Girdwood 	ret = hda_dsp_ctrl_init_chip(sdev, true);
680747503b1SLiam Girdwood 	if (ret < 0) {
681747503b1SLiam Girdwood 		dev_err(sdev->dev,
682747503b1SLiam Girdwood 			"error: failed to start controller after resume\n");
6831372c768SKai Vehmanen 		goto cleanup;
684747503b1SLiam Girdwood 	}
685747503b1SLiam Girdwood 
686fd15f2f5SRander Wang #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
687fd15f2f5SRander Wang 	/* check jack status */
68831ba0c07SKai-Heng Feng 	if (runtime_resume) {
68931ba0c07SKai-Heng Feng 		hda_codec_jack_wake_enable(sdev, false);
690ef4d764cSKai-Heng Feng 		if (sdev->system_suspend_target == SOF_SUSPEND_NONE)
691fd15f2f5SRander Wang 			hda_codec_jack_check(sdev);
69231ba0c07SKai-Heng Feng 	}
6936aa232e1SRander Wang 
6946aa232e1SRander Wang 	/* turn off the links that were off before suspend */
6956aa232e1SRander Wang 	list_for_each_entry(hlink, &bus->hlink_list, list) {
6966aa232e1SRander Wang 		if (!hlink->ref_count)
6976aa232e1SRander Wang 			snd_hdac_ext_bus_link_power_down(hlink);
6986aa232e1SRander Wang 	}
6996aa232e1SRander Wang 
7006aa232e1SRander Wang 	/* check dma status and clean up CORB/RIRB buffers */
7016aa232e1SRander Wang 	if (!bus->cmd_dma_state)
7026aa232e1SRander Wang 		snd_hdac_bus_stop_cmd_io(bus);
70324b6ff68SZhu Yingjiang #endif
704747503b1SLiam Girdwood 
705747503b1SLiam Girdwood 	/* enable ppcap interrupt */
706747503b1SLiam Girdwood 	hda_dsp_ctrl_ppcap_enable(sdev, true);
707747503b1SLiam Girdwood 	hda_dsp_ctrl_ppcap_int_enable(sdev, true);
708747503b1SLiam Girdwood 
7091372c768SKai Vehmanen cleanup:
7101372c768SKai Vehmanen 	/* display codec can powered off after controller init */
7111372c768SKai Vehmanen 	hda_codec_i915_display_power(sdev, false);
7121372c768SKai Vehmanen 
713747503b1SLiam Girdwood 	return 0;
714747503b1SLiam Girdwood }
715747503b1SLiam Girdwood 
716747503b1SLiam Girdwood int hda_dsp_resume(struct snd_sof_dev *sdev)
717747503b1SLiam Girdwood {
71816299326SKeyon Jie 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
71966e40876SKeyon Jie 	struct pci_dev *pci = to_pci_dev(sdev->dev);
72061e285caSRanjani Sridharan 	const struct sof_dsp_power_state target_state = {
72161e285caSRanjani Sridharan 		.state = SOF_DSP_PM_D0,
72261e285caSRanjani Sridharan 		.substate = SOF_HDA_DSP_PM_D0I0,
72361e285caSRanjani Sridharan 	};
724195f1019SMarcin Rajwa #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
725195f1019SMarcin Rajwa 	struct hdac_bus *bus = sof_to_bus(sdev);
726195f1019SMarcin Rajwa 	struct hdac_ext_link *hlink = NULL;
727195f1019SMarcin Rajwa #endif
72861e285caSRanjani Sridharan 	int ret;
72966e40876SKeyon Jie 
73061e285caSRanjani Sridharan 	/* resume from D0I3 */
73161e285caSRanjani Sridharan 	if (sdev->dsp_power_state.state == SOF_DSP_PM_D0) {
732195f1019SMarcin Rajwa #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
733195f1019SMarcin Rajwa 		/* power up links that were active before suspend */
734195f1019SMarcin Rajwa 		list_for_each_entry(hlink, &bus->hlink_list, list) {
735195f1019SMarcin Rajwa 			if (hlink->ref_count) {
736195f1019SMarcin Rajwa 				ret = snd_hdac_ext_bus_link_power_up(hlink);
737195f1019SMarcin Rajwa 				if (ret < 0) {
738195f1019SMarcin Rajwa 					dev_dbg(sdev->dev,
739ce1f55baSCurtis Malainey 						"error %d in %s: failed to power up links",
740195f1019SMarcin Rajwa 						ret, __func__);
741195f1019SMarcin Rajwa 					return ret;
742195f1019SMarcin Rajwa 				}
743195f1019SMarcin Rajwa 			}
744195f1019SMarcin Rajwa 		}
745195f1019SMarcin Rajwa 
746195f1019SMarcin Rajwa 		/* set up CORB/RIRB buffers if was on before suspend */
747195f1019SMarcin Rajwa 		if (bus->cmd_dma_state)
748195f1019SMarcin Rajwa 			snd_hdac_bus_init_cmd_io(bus);
749195f1019SMarcin Rajwa #endif
750195f1019SMarcin Rajwa 
75161e285caSRanjani Sridharan 		/* Set DSP power state */
752787c5214SRanjani Sridharan 		ret = snd_sof_dsp_set_power_state(sdev, &target_state);
75361e285caSRanjani Sridharan 		if (ret < 0) {
75461e285caSRanjani Sridharan 			dev_err(sdev->dev, "error: setting dsp state %d substate %d\n",
75561e285caSRanjani Sridharan 				target_state.state, target_state.substate);
75661e285caSRanjani Sridharan 			return ret;
75761e285caSRanjani Sridharan 		}
75861e285caSRanjani Sridharan 
75916299326SKeyon Jie 		/* restore L1SEN bit */
76016299326SKeyon Jie 		if (hda->l1_support_changed)
76116299326SKeyon Jie 			snd_sof_dsp_update_bits(sdev, HDA_DSP_HDA_BAR,
76216299326SKeyon Jie 						HDA_VS_INTEL_EM2,
76316299326SKeyon Jie 						HDA_VS_INTEL_EM2_L1SEN, 0);
76416299326SKeyon Jie 
76566e40876SKeyon Jie 		/* restore and disable the system wakeup */
76666e40876SKeyon Jie 		pci_restore_state(pci);
76766e40876SKeyon Jie 		disable_irq_wake(pci->irq);
76866e40876SKeyon Jie 		return 0;
76966e40876SKeyon Jie 	}
77066e40876SKeyon Jie 
771747503b1SLiam Girdwood 	/* init hda controller. DSP cores will be powered up during fw boot */
77261e285caSRanjani Sridharan 	ret = hda_resume(sdev, false);
77361e285caSRanjani Sridharan 	if (ret < 0)
77461e285caSRanjani Sridharan 		return ret;
77561e285caSRanjani Sridharan 
776787c5214SRanjani Sridharan 	return snd_sof_dsp_set_power_state(sdev, &target_state);
777747503b1SLiam Girdwood }
778747503b1SLiam Girdwood 
779747503b1SLiam Girdwood int hda_dsp_runtime_resume(struct snd_sof_dev *sdev)
780747503b1SLiam Girdwood {
78161e285caSRanjani Sridharan 	const struct sof_dsp_power_state target_state = {
78261e285caSRanjani Sridharan 		.state = SOF_DSP_PM_D0,
78361e285caSRanjani Sridharan 	};
78461e285caSRanjani Sridharan 	int ret;
78561e285caSRanjani Sridharan 
786747503b1SLiam Girdwood 	/* init hda controller. DSP cores will be powered up during fw boot */
78761e285caSRanjani Sridharan 	ret = hda_resume(sdev, true);
78861e285caSRanjani Sridharan 	if (ret < 0)
78961e285caSRanjani Sridharan 		return ret;
79061e285caSRanjani Sridharan 
791787c5214SRanjani Sridharan 	return snd_sof_dsp_set_power_state(sdev, &target_state);
792747503b1SLiam Girdwood }
793747503b1SLiam Girdwood 
79487a6fe80SKai Vehmanen int hda_dsp_runtime_idle(struct snd_sof_dev *sdev)
79587a6fe80SKai Vehmanen {
79687a6fe80SKai Vehmanen 	struct hdac_bus *hbus = sof_to_bus(sdev);
79787a6fe80SKai Vehmanen 
79887a6fe80SKai Vehmanen 	if (hbus->codec_powered) {
79987a6fe80SKai Vehmanen 		dev_dbg(sdev->dev, "some codecs still powered (%08X), not idle\n",
80087a6fe80SKai Vehmanen 			(unsigned int)hbus->codec_powered);
80187a6fe80SKai Vehmanen 		return -EBUSY;
80287a6fe80SKai Vehmanen 	}
80387a6fe80SKai Vehmanen 
80487a6fe80SKai Vehmanen 	return 0;
80587a6fe80SKai Vehmanen }
80687a6fe80SKai Vehmanen 
8071c38c922SFred Oh int hda_dsp_runtime_suspend(struct snd_sof_dev *sdev)
808747503b1SLiam Girdwood {
8090084364dSRanjani Sridharan 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
81061e285caSRanjani Sridharan 	const struct sof_dsp_power_state target_state = {
81161e285caSRanjani Sridharan 		.state = SOF_DSP_PM_D3,
81261e285caSRanjani Sridharan 	};
81361e285caSRanjani Sridharan 	int ret;
81461e285caSRanjani Sridharan 
8150084364dSRanjani Sridharan 	/* cancel any attempt for DSP D0I3 */
8160084364dSRanjani Sridharan 	cancel_delayed_work_sync(&hda->d0i3_work);
8170084364dSRanjani Sridharan 
818747503b1SLiam Girdwood 	/* stop hda controller and power dsp off */
81961e285caSRanjani Sridharan 	ret = hda_suspend(sdev, true);
82061e285caSRanjani Sridharan 	if (ret < 0)
82161e285caSRanjani Sridharan 		return ret;
82261e285caSRanjani Sridharan 
823787c5214SRanjani Sridharan 	return snd_sof_dsp_set_power_state(sdev, &target_state);
824747503b1SLiam Girdwood }
825747503b1SLiam Girdwood 
82661e285caSRanjani Sridharan int hda_dsp_suspend(struct snd_sof_dev *sdev, u32 target_state)
827747503b1SLiam Girdwood {
82816299326SKeyon Jie 	struct sof_intel_hda_dev *hda = sdev->pdata->hw_pdata;
829747503b1SLiam Girdwood 	struct hdac_bus *bus = sof_to_bus(sdev);
83066e40876SKeyon Jie 	struct pci_dev *pci = to_pci_dev(sdev->dev);
83161e285caSRanjani Sridharan 	const struct sof_dsp_power_state target_dsp_state = {
83261e285caSRanjani Sridharan 		.state = target_state,
83361e285caSRanjani Sridharan 		.substate = target_state == SOF_DSP_PM_D0 ?
83461e285caSRanjani Sridharan 				SOF_HDA_DSP_PM_D0I3 : 0,
83561e285caSRanjani Sridharan 	};
836747503b1SLiam Girdwood 	int ret;
837747503b1SLiam Girdwood 
83863e51fd3SRanjani Sridharan 	/* cancel any attempt for DSP D0I3 */
83963e51fd3SRanjani Sridharan 	cancel_delayed_work_sync(&hda->d0i3_work);
84063e51fd3SRanjani Sridharan 
84161e285caSRanjani Sridharan 	if (target_state == SOF_DSP_PM_D0) {
84261e285caSRanjani Sridharan 		/* Set DSP power state */
843787c5214SRanjani Sridharan 		ret = snd_sof_dsp_set_power_state(sdev, &target_dsp_state);
84461e285caSRanjani Sridharan 		if (ret < 0) {
84561e285caSRanjani Sridharan 			dev_err(sdev->dev, "error: setting dsp state %d substate %d\n",
84661e285caSRanjani Sridharan 				target_dsp_state.state,
84761e285caSRanjani Sridharan 				target_dsp_state.substate);
84861e285caSRanjani Sridharan 			return ret;
84961e285caSRanjani Sridharan 		}
85061e285caSRanjani Sridharan 
85116299326SKeyon Jie 		/* enable L1SEN to make sure the system can enter S0Ix */
85216299326SKeyon Jie 		hda->l1_support_changed =
85316299326SKeyon Jie 			snd_sof_dsp_update_bits(sdev, HDA_DSP_HDA_BAR,
85416299326SKeyon Jie 						HDA_VS_INTEL_EM2,
85516299326SKeyon Jie 						HDA_VS_INTEL_EM2_L1SEN,
85616299326SKeyon Jie 						HDA_VS_INTEL_EM2_L1SEN);
85716299326SKeyon Jie 
858195f1019SMarcin Rajwa #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
859195f1019SMarcin Rajwa 		/* stop the CORB/RIRB DMA if it is On */
860195f1019SMarcin Rajwa 		if (bus->cmd_dma_state)
861195f1019SMarcin Rajwa 			snd_hdac_bus_stop_cmd_io(bus);
862195f1019SMarcin Rajwa 
863195f1019SMarcin Rajwa 		/* no link can be powered in s0ix state */
864195f1019SMarcin Rajwa 		ret = snd_hdac_ext_bus_link_power_down_all(bus);
865195f1019SMarcin Rajwa 		if (ret < 0) {
866195f1019SMarcin Rajwa 			dev_dbg(sdev->dev,
867195f1019SMarcin Rajwa 				"error %d in %s: failed to power down links",
868195f1019SMarcin Rajwa 				ret, __func__);
869195f1019SMarcin Rajwa 			return ret;
870195f1019SMarcin Rajwa 		}
871195f1019SMarcin Rajwa #endif
872195f1019SMarcin Rajwa 
87366e40876SKeyon Jie 		/* enable the system waking up via IPC IRQ */
87466e40876SKeyon Jie 		enable_irq_wake(pci->irq);
87566e40876SKeyon Jie 		pci_save_state(pci);
87666e40876SKeyon Jie 		return 0;
87766e40876SKeyon Jie 	}
87866e40876SKeyon Jie 
879747503b1SLiam Girdwood 	/* stop hda controller and power dsp off */
8801c38c922SFred Oh 	ret = hda_suspend(sdev, false);
881747503b1SLiam Girdwood 	if (ret < 0) {
882747503b1SLiam Girdwood 		dev_err(bus->dev, "error: suspending dsp\n");
883747503b1SLiam Girdwood 		return ret;
884747503b1SLiam Girdwood 	}
885747503b1SLiam Girdwood 
886787c5214SRanjani Sridharan 	return snd_sof_dsp_set_power_state(sdev, &target_dsp_state);
887747503b1SLiam Girdwood }
888ed3baacdSRanjani Sridharan 
88922aa9e02SLibin Yang int hda_dsp_shutdown(struct snd_sof_dev *sdev)
89022aa9e02SLibin Yang {
89122aa9e02SLibin Yang 	sdev->system_suspend_target = SOF_SUSPEND_S3;
89222aa9e02SLibin Yang 	return snd_sof_suspend(sdev->dev);
89322aa9e02SLibin Yang }
89422aa9e02SLibin Yang 
8957077a07aSRanjani Sridharan int hda_dsp_set_hw_params_upon_resume(struct snd_sof_dev *sdev)
896ed3baacdSRanjani Sridharan {
8977077a07aSRanjani Sridharan #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
898a3ebccb5SKai Vehmanen 	struct hdac_bus *bus = sof_to_bus(sdev);
8997077a07aSRanjani Sridharan 	struct snd_soc_pcm_runtime *rtd;
9007d88b960SPierre-Louis Bossart 	struct hdac_ext_stream *hext_stream;
9017077a07aSRanjani Sridharan 	struct hdac_ext_link *link;
902a3ebccb5SKai Vehmanen 	struct hdac_stream *s;
9037077a07aSRanjani Sridharan 	const char *name;
9047077a07aSRanjani Sridharan 	int stream_tag;
9057077a07aSRanjani Sridharan 
906ed3baacdSRanjani Sridharan 	/* set internal flag for BE */
907ed3baacdSRanjani Sridharan 	list_for_each_entry(s, &bus->stream_list, list) {
9087d88b960SPierre-Louis Bossart 		hext_stream = stream_to_hdac_ext_stream(s);
909a3ebccb5SKai Vehmanen 
9107077a07aSRanjani Sridharan 		/*
911934bf822SRander Wang 		 * clear stream. This should already be taken care for running
912934bf822SRander Wang 		 * streams when the SUSPEND trigger is called. But paused
913934bf822SRander Wang 		 * streams do not get suspended, so this needs to be done
914934bf822SRander Wang 		 * explicitly during suspend.
9157077a07aSRanjani Sridharan 		 */
9167d88b960SPierre-Louis Bossart 		if (hext_stream->link_substream) {
9177d88b960SPierre-Louis Bossart 			rtd = asoc_substream_to_rtd(hext_stream->link_substream);
918be3e8de7SKuninori Morimoto 			name = asoc_rtd_to_codec(rtd, 0)->component->name;
9197077a07aSRanjani Sridharan 			link = snd_hdac_ext_bus_get_link(bus, name);
9207077a07aSRanjani Sridharan 			if (!link)
9217077a07aSRanjani Sridharan 				return -EINVAL;
922810dbea3SRander Wang 
9237d88b960SPierre-Louis Bossart 			hext_stream->link_prepared = 0;
924810dbea3SRander Wang 
9257d88b960SPierre-Louis Bossart 			if (hdac_stream(hext_stream)->direction ==
926810dbea3SRander Wang 				SNDRV_PCM_STREAM_CAPTURE)
927810dbea3SRander Wang 				continue;
928810dbea3SRander Wang 
9297d88b960SPierre-Louis Bossart 			stream_tag = hdac_stream(hext_stream)->stream_tag;
9307077a07aSRanjani Sridharan 			snd_hdac_ext_link_clear_stream_id(link, stream_tag);
931a3ebccb5SKai Vehmanen 		}
932ed3baacdSRanjani Sridharan 	}
9337077a07aSRanjani Sridharan #endif
9347077a07aSRanjani Sridharan 	return 0;
935ed3baacdSRanjani Sridharan }
93663e51fd3SRanjani Sridharan 
93763e51fd3SRanjani Sridharan void hda_dsp_d0i3_work(struct work_struct *work)
93863e51fd3SRanjani Sridharan {
93963e51fd3SRanjani Sridharan 	struct sof_intel_hda_dev *hdev = container_of(work,
94063e51fd3SRanjani Sridharan 						      struct sof_intel_hda_dev,
94163e51fd3SRanjani Sridharan 						      d0i3_work.work);
94263e51fd3SRanjani Sridharan 	struct hdac_bus *bus = &hdev->hbus.core;
94363e51fd3SRanjani Sridharan 	struct snd_sof_dev *sdev = dev_get_drvdata(bus->dev);
944f1bb0235SGuennadi Liakhovetski 	struct sof_dsp_power_state target_state = {
945f1bb0235SGuennadi Liakhovetski 		.state = SOF_DSP_PM_D0,
946f1bb0235SGuennadi Liakhovetski 		.substate = SOF_HDA_DSP_PM_D0I3,
947f1bb0235SGuennadi Liakhovetski 	};
94863e51fd3SRanjani Sridharan 	int ret;
94963e51fd3SRanjani Sridharan 
95063e51fd3SRanjani Sridharan 	/* DSP can enter D0I3 iff only D0I3-compatible streams are active */
951f1bb0235SGuennadi Liakhovetski 	if (!snd_sof_dsp_only_d0i3_compatible_stream_active(sdev))
95263e51fd3SRanjani Sridharan 		/* remain in D0I0 */
95363e51fd3SRanjani Sridharan 		return;
95463e51fd3SRanjani Sridharan 
95563e51fd3SRanjani Sridharan 	/* This can fail but error cannot be propagated */
956787c5214SRanjani Sridharan 	ret = snd_sof_dsp_set_power_state(sdev, &target_state);
95763e51fd3SRanjani Sridharan 	if (ret < 0)
95863e51fd3SRanjani Sridharan 		dev_err_ratelimited(sdev->dev,
95963e51fd3SRanjani Sridharan 				    "error: failed to set DSP state %d substate %d\n",
96063e51fd3SRanjani Sridharan 				    target_state.state, target_state.substate);
96163e51fd3SRanjani Sridharan }
9629cdcbc9fSRanjani Sridharan 
9639cdcbc9fSRanjani Sridharan int hda_dsp_core_get(struct snd_sof_dev *sdev, int core)
9649cdcbc9fSRanjani Sridharan {
9659cdcbc9fSRanjani Sridharan 	struct sof_ipc_pm_core_config pm_core_config = {
9669cdcbc9fSRanjani Sridharan 		.hdr = {
9679cdcbc9fSRanjani Sridharan 			.cmd = SOF_IPC_GLB_PM_MSG | SOF_IPC_PM_CORE_ENABLE,
9689cdcbc9fSRanjani Sridharan 			.size = sizeof(pm_core_config),
9699cdcbc9fSRanjani Sridharan 		},
9709cdcbc9fSRanjani Sridharan 		.enable_mask = sdev->enabled_cores_mask | BIT(core),
9719cdcbc9fSRanjani Sridharan 	};
9729cdcbc9fSRanjani Sridharan 	int ret, ret1;
9739cdcbc9fSRanjani Sridharan 
9749cdcbc9fSRanjani Sridharan 	/* power up core */
9759cdcbc9fSRanjani Sridharan 	ret = hda_dsp_enable_core(sdev, BIT(core));
9769cdcbc9fSRanjani Sridharan 	if (ret < 0) {
9779cdcbc9fSRanjani Sridharan 		dev_err(sdev->dev, "failed to power up core %d with err: %d\n",
9789cdcbc9fSRanjani Sridharan 			core, ret);
9799cdcbc9fSRanjani Sridharan 		return ret;
9809cdcbc9fSRanjani Sridharan 	}
9819cdcbc9fSRanjani Sridharan 
9829cdcbc9fSRanjani Sridharan 	/* No need to send IPC for primary core or if FW boot is not complete */
9839cdcbc9fSRanjani Sridharan 	if (sdev->fw_state != SOF_FW_BOOT_COMPLETE || core == SOF_DSP_PRIMARY_CORE)
9849cdcbc9fSRanjani Sridharan 		return 0;
9859cdcbc9fSRanjani Sridharan 
9869cdcbc9fSRanjani Sridharan 	/* Now notify DSP for secondary cores */
987*2a51c0f8SPeter Ujfalusi 	ret = sof_ipc_tx_message(sdev->ipc, &pm_core_config, sizeof(pm_core_config),
9889cdcbc9fSRanjani Sridharan 				 &pm_core_config, sizeof(pm_core_config));
9899cdcbc9fSRanjani Sridharan 	if (ret < 0) {
9909cdcbc9fSRanjani Sridharan 		dev_err(sdev->dev, "failed to enable secondary core '%d' failed with %d\n",
9919cdcbc9fSRanjani Sridharan 			core, ret);
9929cdcbc9fSRanjani Sridharan 		goto power_down;
9939cdcbc9fSRanjani Sridharan 	}
9949cdcbc9fSRanjani Sridharan 
9959cdcbc9fSRanjani Sridharan 	return ret;
9969cdcbc9fSRanjani Sridharan 
9979cdcbc9fSRanjani Sridharan power_down:
9989cdcbc9fSRanjani Sridharan 	/* power down core if it is host managed and return the original error if this fails too */
9999cdcbc9fSRanjani Sridharan 	ret1 = hda_dsp_core_reset_power_down(sdev, BIT(core));
10009cdcbc9fSRanjani Sridharan 	if (ret1 < 0)
10019cdcbc9fSRanjani Sridharan 		dev_err(sdev->dev, "failed to power down core: %d with err: %d\n", core, ret1);
10029cdcbc9fSRanjani Sridharan 
10039cdcbc9fSRanjani Sridharan 	return ret;
10049cdcbc9fSRanjani Sridharan }
1005