Commit a719afe6 authored by Lucas Tanure's avatar Lucas Tanure Committed by Mark Brown

ASoC: amd: vangogh: use for_each_rtd_components instead of for

To iterate over components use for_each_rtd_components
And compare to component name, so asoc_rtd_to_codec and the dai code can
be removed
Signed-off-by: default avatarLucas Tanure <lucas.tanure@collabora.com>
Link: https://lore.kernel.org/r/20230217110850.1045250-6-lucas.tanure@collabora.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 88f5870d
...@@ -153,35 +153,37 @@ static int acp5x_cs35l41_hw_params(struct snd_pcm_substream *substream, ...@@ -153,35 +153,37 @@ static int acp5x_cs35l41_hw_params(struct snd_pcm_substream *substream,
struct snd_pcm_hw_params *params) struct snd_pcm_hw_params *params)
{ {
struct snd_soc_pcm_runtime *rtd = asoc_substream_to_rtd(substream); struct snd_soc_pcm_runtime *rtd = asoc_substream_to_rtd(substream);
unsigned int num_codecs = rtd->dai_link->num_codecs; unsigned int bclk, rate = params_rate(params);
struct snd_soc_card *card = rtd->card; struct snd_soc_component *comp;
struct snd_soc_dai *dai;
unsigned int bclk_val;
int ret, i; int ret, i;
ret = 0; switch (rate) {
for (i = 0; i < num_codecs; i++) {
dai = asoc_rtd_to_codec(rtd, i);
if (strcmp(dai->name, "cs35l41-pcm") == 0) {
switch (params_rate(params)) {
case 48000: case 48000:
bclk_val = 1536000; bclk = 1536000;
break; break;
default: default:
dev_err(card->dev, "Invalid Samplerate:0x%x\n", bclk = 0;
params_rate(params)); break;
}
for_each_rtd_components(rtd, i, comp) {
if (!(strcmp(comp->name, "spi-VLV1776:00")) ||
!(strcmp(comp->name, "spi-VLV1776:01"))) {
if (!bclk) {
dev_err(comp->dev, "Invalid sample rate: 0x%x\n", rate);
return -EINVAL; return -EINVAL;
} }
ret = snd_soc_component_set_sysclk(dai->component, 0, 0,
bclk_val, SND_SOC_CLOCK_IN); ret = snd_soc_component_set_sysclk(comp, 0, 0, bclk, SND_SOC_CLOCK_IN);
if (ret < 0) { if (ret) {
dev_err(card->dev, "failed to set sysclk for CS35l41 dai\n"); dev_err(comp->dev, "failed to set SYSCLK: %d\n", ret);
return ret; return ret;
} }
} }
} }
return ret; return 0;
} }
static const struct snd_soc_ops acp5x_8821_ops = { static const struct snd_soc_ops acp5x_8821_ops = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment