Skip to content

Commit

Permalink
ASoC: SOF: ipc4-topology: Improve the information in prepare_copier p…
Browse files Browse the repository at this point in the history
…rints

It is useful to know the explicit type and if the copier (host/dai) is
configured to use ChainDMA or not and also the stream_tag for the host
copier.

Change the prints to carry more information for debugging purposes.

Signed-off-by: Peter Ujfalusi <[email protected]>
  • Loading branch information
ujfalusi committed Jan 8, 2025
1 parent b6d3bb5 commit d301dbd
Showing 1 changed file with 15 additions and 7 deletions.
22 changes: 15 additions & 7 deletions sound/soc/sof/ipc4-topology.c
Original file line number Diff line number Diff line change
Expand Up @@ -1821,15 +1821,19 @@ sof_ipc4_prepare_copier_module(struct snd_sof_widget *swidget,
bool single_output_bitdepth;
int i;

dev_dbg(sdev->dev, "copier %s, type %d", swidget->widget->name, swidget->id);

switch (swidget->id) {
case snd_soc_dapm_aif_in:
case snd_soc_dapm_aif_out:
{
struct snd_sof_widget *pipe_widget = swidget->spipe->pipe_widget;
struct sof_ipc4_pipeline *pipeline = pipe_widget->private;
struct sof_ipc4_gtw_attributes *gtw_attr;
struct snd_sof_widget *pipe_widget;
struct sof_ipc4_pipeline *pipeline;

dev_dbg(sdev->dev,
"Host copier %s, type %d, , ChainDMA: %s, stream_tag: %d\n",
swidget->widget->name, swidget->id,
str_yes_no(pipeline->use_chain_dma),
platform_params->stream_tag);

/* parse the deep buffer dma size */
ret = sof_update_ipc_object(scomp, &deep_buffer_dma_ms,
Expand All @@ -1846,9 +1850,6 @@ sof_ipc4_prepare_copier_module(struct snd_sof_widget *swidget,
copier_data = &ipc4_copier->data;
available_fmt = &ipc4_copier->available_fmt;

pipe_widget = swidget->spipe->pipe_widget;
pipeline = pipe_widget->private;

if (pipeline->use_chain_dma) {
u32 host_dma_id;
u32 fifo_size;
Expand Down Expand Up @@ -1902,6 +1903,10 @@ sof_ipc4_prepare_copier_module(struct snd_sof_widget *swidget,
struct snd_sof_widget *pipe_widget = swidget->spipe->pipe_widget;
struct sof_ipc4_pipeline *pipeline = pipe_widget->private;

dev_dbg(sdev->dev, "Dai copier %s, type %d, ChainDMA: %s\n",
swidget->widget->name, swidget->id,
str_yes_no(pipeline->use_chain_dma));

if (pipeline->use_chain_dma)
return 0;

Expand Down Expand Up @@ -1935,6 +1940,9 @@ sof_ipc4_prepare_copier_module(struct snd_sof_widget *swidget,
}
case snd_soc_dapm_buffer:
{
dev_dbg(sdev->dev, "Module copier %s, type %d\n",
swidget->widget->name, swidget->id);

ipc4_copier = (struct sof_ipc4_copier *)swidget->private;
copier_data = &ipc4_copier->data;
available_fmt = &ipc4_copier->available_fmt;
Expand Down

0 comments on commit d301dbd

Please sign in to comment.