ASoC: codec2codec: run callbacks in order
authorJerome Brunet <jbrunet@baylibre.com>
Thu, 25 Jul 2019 16:59:44 +0000 (18:59 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 25 Jul 2019 17:42:15 +0000 (18:42 +0100)
When handling dai_link events on codec to codec links, run all .startup()
callbacks on sinks and sources before running any .hw_params(). Same goes
for hw_free() and shutdown(). This is closer to the behavior of regular
dai links

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Link: https://lore.kernel.org/r/20190725165949.29699-2-jbrunet@baylibre.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-dapm.c

index 1d04612601ad7e404d47a98e25bbeff2227feab4..034b31fd2ecb9b795e240b176789b424ded04589 100644 (file)
@@ -3835,11 +3835,6 @@ static int snd_soc_dai_link_event(struct snd_soc_dapm_widget *w,
                                goto out;
                        }
                        source->active++;
-                       ret = snd_soc_dai_hw_params(source, &substream, params);
-                       if (ret < 0)
-                               goto out;
-
-                       dapm_update_dai_unlocked(&substream, params, source);
                }
 
                substream.stream = SNDRV_PCM_STREAM_PLAYBACK;
@@ -3853,6 +3848,23 @@ static int snd_soc_dai_link_event(struct snd_soc_dapm_widget *w,
                                goto out;
                        }
                        sink->active++;
+               }
+
+               substream.stream = SNDRV_PCM_STREAM_CAPTURE;
+               snd_soc_dapm_widget_for_each_source_path(w, path) {
+                       source = path->source->priv;
+
+                       ret = snd_soc_dai_hw_params(source, &substream, params);
+                       if (ret < 0)
+                               goto out;
+
+                       dapm_update_dai_unlocked(&substream, params, source);
+               }
+
+               substream.stream = SNDRV_PCM_STREAM_PLAYBACK;
+               snd_soc_dapm_widget_for_each_sink_path(w, path) {
+                       sink = path->sink->priv;
+
                        ret = snd_soc_dai_hw_params(sink, &substream, params);
                        if (ret < 0)
                                goto out;
@@ -3889,9 +3901,18 @@ static int snd_soc_dai_link_event(struct snd_soc_dapm_widget *w,
                substream.stream = SNDRV_PCM_STREAM_CAPTURE;
                snd_soc_dapm_widget_for_each_source_path(w, path) {
                        source = path->source->priv;
-
                        snd_soc_dai_hw_free(source, &substream);
+               }
+
+               substream.stream = SNDRV_PCM_STREAM_PLAYBACK;
+               snd_soc_dapm_widget_for_each_sink_path(w, path) {
+                       sink = path->sink->priv;
+                       snd_soc_dai_hw_free(sink, &substream);
+               }
 
+               substream.stream = SNDRV_PCM_STREAM_CAPTURE;
+               snd_soc_dapm_widget_for_each_source_path(w, path) {
+                       source = path->source->priv;
                        source->active--;
                        snd_soc_dai_shutdown(source, &substream);
                }
@@ -3899,9 +3920,6 @@ static int snd_soc_dai_link_event(struct snd_soc_dapm_widget *w,
                substream.stream = SNDRV_PCM_STREAM_PLAYBACK;
                snd_soc_dapm_widget_for_each_sink_path(w, path) {
                        sink = path->sink->priv;
-
-                       snd_soc_dai_hw_free(sink, &substream);
-
                        sink->active--;
                        snd_soc_dai_shutdown(sink, &substream);
                }