ASoC: codec2codec: run callbacks in order
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>
This commit is contained in:
parent
6535e831b4
commit
68c907f10c
|
@ -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);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue