ASoC: SOF: Intel: hda-dai: enhance debug messages
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Fri, 10 Jun 2022 21:45:04 +0000 (16:45 -0500)
committerMark Brown <broonie@kernel.org>
Tue, 14 Jun 2022 08:14:17 +0000 (09:14 +0100)
The same message was added twice for dai and link_dma, remove the
latter one and add dai name and direction to better understand
problematic sequences.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com>
Link: https://lore.kernel.org/r/20220610214504.42974-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/intel/hda-dai.c

index 228079a..70721de 100644 (file)
@@ -270,7 +270,6 @@ static int hda_link_dma_trigger(struct snd_pcm_substream *substream, int cmd)
        struct hdac_ext_stream *hext_stream = snd_soc_dai_get_dma_data(cpu_dai, substream);
        int ret;
 
-       dev_dbg(cpu_dai->dev, "%s: cmd=%d\n", __func__, cmd);
        if (!hext_stream)
                return 0;
 
@@ -420,13 +419,15 @@ static int ipc3_hda_dai_trigger(struct snd_pcm_substream *substream,
        struct snd_soc_dapm_widget *w;
        int ret;
 
+       dev_dbg(dai->dev, "%s: cmd=%d dai %s direction %d\n", __func__, cmd,
+               dai->name, substream->stream);
+
        ret = hda_link_dma_trigger(substream, cmd);
        if (ret < 0)
                return ret;
 
        w = snd_soc_dai_get_widget(dai, substream->stream);
 
-       dev_dbg(dai->dev, "%s: cmd=%d\n", __func__, cmd);
        switch (cmd) {
        case SNDRV_PCM_TRIGGER_SUSPEND:
        case SNDRV_PCM_TRIGGER_STOP: