|
Jiri Slaby |
6616e5 |
From: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Date: Wed, 22 Mar 2023 10:55:38 +0200
|
|
Jiri Slaby |
6616e5 |
Subject: [PATCH] ASoC: SOF: ipc4: Ensure DSP is in D0I0 during
|
|
Jiri Slaby |
6616e5 |
sof_ipc4_set_get_data()
|
|
Jiri Slaby |
6616e5 |
MIME-Version: 1.0
|
|
Jiri Slaby |
6616e5 |
Content-Type: text/plain; charset=UTF-8
|
|
Jiri Slaby |
6616e5 |
Content-Transfer-Encoding: 8bit
|
|
Jiri Slaby |
6616e5 |
References: bsc#1012628
|
|
Jiri Slaby |
6616e5 |
Patch-mainline: 6.2.11
|
|
Jiri Slaby |
6616e5 |
Git-commit: e51f49512d98783b90799c9cc2002895ec3aa0eb
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
[ Upstream commit e51f49512d98783b90799c9cc2002895ec3aa0eb ]
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
The set_get_data() IPC op bypasses the check for the no_pm flag as done
|
|
Jiri Slaby |
6616e5 |
with the regular IPC tx_msg op. Since set_get_data should be performed
|
|
Jiri Slaby |
6616e5 |
when the DSP is in D0I0, set the DSP power state to D0I0 before sending
|
|
Jiri Slaby |
6616e5 |
the IPC's in sof_ipc4_set_get_data().
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
Fixes: ceb89acc4dc8 ("ASoC: SOF: ipc4: Add support for mandatory message handling functionality")
|
|
Jiri Slaby |
6616e5 |
Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
|
|
Jiri Slaby |
6616e5 |
Link: https://lore.kernel.org/r/20230322085538.10214-1-peter.ujfalusi@linux.intel.com
|
|
Jiri Slaby |
6616e5 |
Signed-off-by: Mark Brown <broonie@kernel.org>
|
|
Jiri Slaby |
6616e5 |
Signed-off-by: Sasha Levin <sashal@kernel.org>
|
|
Jiri Slaby |
6616e5 |
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
|
|
Jiri Slaby |
6616e5 |
---
|
|
Jiri Slaby |
6616e5 |
sound/soc/sof/ipc4.c | 8 ++++++++
|
|
Jiri Slaby |
6616e5 |
1 file changed, 8 insertions(+)
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
diff --git a/sound/soc/sof/ipc4.c b/sound/soc/sof/ipc4.c
|
|
Jiri Slaby |
6616e5 |
index 74cd7e95..280fc890 100644
|
|
Jiri Slaby |
6616e5 |
--- a/sound/soc/sof/ipc4.c
|
|
Jiri Slaby |
6616e5 |
+++ b/sound/soc/sof/ipc4.c
|
|
Jiri Slaby |
6616e5 |
@@ -393,6 +393,9 @@ static int sof_ipc4_tx_msg(struct snd_sof_dev *sdev, void *msg_data, size_t msg_
|
|
Jiri Slaby |
6616e5 |
static int sof_ipc4_set_get_data(struct snd_sof_dev *sdev, void *data,
|
|
Jiri Slaby |
6616e5 |
size_t payload_bytes, bool set)
|
|
Jiri Slaby |
6616e5 |
{
|
|
Jiri Slaby |
6616e5 |
+ const struct sof_dsp_power_state target_state = {
|
|
Jiri Slaby |
6616e5 |
+ .state = SOF_DSP_PM_D0,
|
|
Jiri Slaby |
6616e5 |
+ };
|
|
Jiri Slaby |
6616e5 |
size_t payload_limit = sdev->ipc->max_payload_size;
|
|
Jiri Slaby |
6616e5 |
struct sof_ipc4_msg *ipc4_msg = data;
|
|
Jiri Slaby |
6616e5 |
struct sof_ipc4_msg tx = {{ 0 }};
|
|
Jiri Slaby |
6616e5 |
@@ -423,6 +426,11 @@ static int sof_ipc4_set_get_data(struct snd_sof_dev *sdev, void *data,
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
tx.extension |= SOF_IPC4_MOD_EXT_MSG_FIRST_BLOCK(1);
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
+ /* ensure the DSP is in D0i0 before sending IPC */
|
|
Jiri Slaby |
6616e5 |
+ ret = snd_sof_dsp_set_power_state(sdev, &target_state);
|
|
Jiri Slaby |
6616e5 |
+ if (ret < 0)
|
|
Jiri Slaby |
6616e5 |
+ return ret;
|
|
Jiri Slaby |
6616e5 |
+
|
|
Jiri Slaby |
6616e5 |
/* Serialise IPC TX */
|
|
Jiri Slaby |
6616e5 |
mutex_lock(&sdev->ipc->tx_mutex);
|
|
Jiri Slaby |
6616e5 |
|
|
Jiri Slaby |
6616e5 |
--
|
|
Jiri Slaby |
6616e5 |
2.35.3
|
|
Jiri Slaby |
6616e5 |
|