diff options
| author | Peter Ujfalusi <peter.ujfalusi@linux.intel.com> | 2023-09-14 15:51:15 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-09-14 15:56:56 +0100 |
| commit | f7d67a9c254829930355d675e989c0dfa884242c (patch) | |
| tree | 2faea327b15d2d0673b79e42bd76215b0c51aa95 /sound/soc/sof/ipc4.c | |
| parent | 353bc9924cb1b7176fdc4ebb3610306398f41c94 (diff) | |
| download | linux-stable-f7d67a9c254829930355d675e989c0dfa884242c.tar.gz linux-stable-f7d67a9c254829930355d675e989c0dfa884242c.tar.bz2 linux-stable-f7d67a9c254829930355d675e989c0dfa884242c.zip | |
ASoC: SOF: ipc4: Dump the payload also when set_get_data fails
Move the out label to dump the message payload when the IPC message fails.
The payload contains important information on what might have caused the
error in firmware.
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20230914125115.30904-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/sof/ipc4.c')
| -rw-r--r-- | sound/soc/sof/ipc4.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/sof/ipc4.c b/sound/soc/sof/ipc4.c index ab6eddd91bb7..9744627d6978 100644 --- a/sound/soc/sof/ipc4.c +++ b/sound/soc/sof/ipc4.c @@ -513,10 +513,10 @@ static int sof_ipc4_set_get_data(struct snd_sof_dev *sdev, void *data, if (!set && payload_bytes != offset) ipc4_msg->data_size = offset; +out: if (sof_debug_check_flag(SOF_DBG_DUMP_IPC_MESSAGE_PAYLOAD)) sof_ipc4_dump_payload(sdev, ipc4_msg->data_ptr, ipc4_msg->data_size); -out: mutex_unlock(&sdev->ipc->tx_mutex); return ret; |