提交 4d8adccb 编写于 作者: S Subhransu S. Prusty 提交者: Mark Brown

ASoC: Intel: Skylake: Fix to fill all sink/source pipe params

Currently params only for first copier widget identified in the
source/sink path is queried from NHLT. In the dapm route the
playback/capture widget may be connected to more than one copier
widget. This patch adds return check to return only for any error
case.
Signed-off-by: NSubhransu S. Prusty <subhransu.s.prusty@intel.com>
Signed-off-by: NJeeja KP <jeeja.kp@intel.com>
Signed-off-by: NVinod Koul <vinod.koul@intel.com>
Signed-off-by: NMark Brown <broonie@kernel.org>
上级 f0900eb2
...@@ -944,48 +944,60 @@ static int skl_tplg_be_set_src_pipe_params(struct snd_soc_dai *dai, ...@@ -944,48 +944,60 @@ static int skl_tplg_be_set_src_pipe_params(struct snd_soc_dai *dai,
struct skl_pipe_params *params) struct skl_pipe_params *params)
{ {
struct snd_soc_dapm_path *p; struct snd_soc_dapm_path *p;
int ret = -EIO;
snd_soc_dapm_widget_for_each_source_path(w, p) { snd_soc_dapm_widget_for_each_source_path(w, p) {
if (p->connect && is_skl_dsp_widget_type(p->source) && if (p->connect && is_skl_dsp_widget_type(p->source) &&
p->source->priv) { p->source->priv) {
if (!p->source->power) if (!p->source->power) {
return skl_tplg_be_fill_pipe_params( ret = skl_tplg_be_fill_pipe_params(
dai, p->source->priv, dai, p->source->priv,
params); params);
else if (ret < 0)
return ret;
} else {
return -EBUSY; return -EBUSY;
}
} else { } else {
return skl_tplg_be_set_src_pipe_params( ret = skl_tplg_be_set_src_pipe_params(
dai, p->source, params); dai, p->source, params);
if (ret < 0)
return ret;
} }
} }
return -EIO; return ret;
} }
static int skl_tplg_be_set_sink_pipe_params(struct snd_soc_dai *dai, static int skl_tplg_be_set_sink_pipe_params(struct snd_soc_dai *dai,
struct snd_soc_dapm_widget *w, struct skl_pipe_params *params) struct snd_soc_dapm_widget *w, struct skl_pipe_params *params)
{ {
struct snd_soc_dapm_path *p = NULL; struct snd_soc_dapm_path *p = NULL;
int ret = -EIO;
snd_soc_dapm_widget_for_each_sink_path(w, p) { snd_soc_dapm_widget_for_each_sink_path(w, p) {
if (p->connect && is_skl_dsp_widget_type(p->sink) && if (p->connect && is_skl_dsp_widget_type(p->sink) &&
p->sink->priv) { p->sink->priv) {
if (!p->sink->power) if (!p->sink->power) {
return skl_tplg_be_fill_pipe_params( ret = skl_tplg_be_fill_pipe_params(
dai, p->sink->priv, params); dai, p->sink->priv, params);
else if (ret < 0)
return ret;
} else {
return -EBUSY; return -EBUSY;
}
} else { } else {
return skl_tplg_be_set_sink_pipe_params( ret = skl_tplg_be_set_sink_pipe_params(
dai, p->sink, params); dai, p->sink, params);
if (ret < 0)
return ret;
} }
} }
return -EIO; return ret;
} }
/* /*
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册