diff mbox series

[2/2] ASoC: soc-pcm.c: factorize CPU/Codec validation check on soc_get_playback_capture()

Message ID 87fs2qg698.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
State New, archived
Headers show
Series ASoC: CPU/Codec connection cleanup - step1 | expand

Commit Message

Kuninori Morimoto Oct. 4, 2023, 8:07 a.m. UTC
Current soc_get_playback_capture() is checking CPU/Codec validation.
But it is using different operation for each 1:N, N:N, N:M connections,
thus it is very complex.

Therefore, there was omission of check at 1:N, N:N, N:M connections.
It was handles as "valid" eventhough it was just "at least one of
CPU/Codec pair was valid", and off course it was wrong.
It is fixed and handled as "valid" when "all CPUs and Codecs were valid"
today.

Because it checks all CPUs/Codecs today, we no longer need to think
about CPU/Codec connection pair style.
This patch cleanup CPU/Codec validation check.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 sound/soc/soc-pcm.c | 43 ++++++++++++++-----------------------------
 1 file changed, 14 insertions(+), 29 deletions(-)
diff mbox series

Patch

diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c
index a45c0cf0fa14..1e7925b93689 100644
--- a/sound/soc/soc-pcm.c
+++ b/sound/soc/soc-pcm.c
@@ -2820,43 +2820,28 @@  static int soc_get_playback_capture(struct snd_soc_pcm_runtime *rtd,
 			}
 		}
 	} else {
-		struct snd_soc_dai *codec_dai;
+		struct snd_soc_dai *dai;
 
 		/* Adapt stream for codec2codec links */
 		int cpu_capture  = snd_soc_get_stream_cpu(dai_link, SNDRV_PCM_STREAM_CAPTURE);
 		int cpu_playback = snd_soc_get_stream_cpu(dai_link, SNDRV_PCM_STREAM_PLAYBACK);
 
-		has_playback = (dai_link->num_codecs > 0);
-		has_capture  = (dai_link->num_codecs > 0);
-		for_each_rtd_codec_dais(rtd, i, codec_dai) {
-			if (dai_link->num_cpus == 1) {
-				cpu_dai = snd_soc_rtd_to_cpu(rtd, 0);
-			} else if (dai_link->num_cpus == dai_link->num_codecs) {
-				cpu_dai = snd_soc_rtd_to_cpu(rtd, i);
-			} else if (rtd->dai_link->num_codecs > rtd->dai_link->num_cpus) {
-				int cpu_id;
-
-				if (!rtd->dai_link->codec_ch_maps) {
-					dev_err(rtd->card->dev, "%s: no codec channel mapping table provided\n",
-						__func__);
-					return -EINVAL;
-				}
+		has_playback =
+		has_capture  = (dai_link->num_cpus > 0 && dai_link->num_codecs > 0);
 
-				cpu_id = rtd->dai_link->codec_ch_maps[i].connected_cpu_id;
-				cpu_dai = snd_soc_rtd_to_cpu(rtd, cpu_id);
-			} else {
-				dev_err(rtd->card->dev,
-					"%s codec number %d < cpu number %d is not supported\n",
-					__func__, rtd->dai_link->num_codecs,
-					rtd->dai_link->num_cpus);
-				return -EINVAL;
-			}
+		/* CPU validation check */
+		for_each_rtd_cpu_dais(rtd, i, dai) {
+			if (!snd_soc_dai_stream_valid(dai, cpu_playback))
+				has_playback = 0;
+			if (!snd_soc_dai_stream_valid(dai, cpu_capture))
+				has_capture = 0;
+		}
 
-			if (!(snd_soc_dai_stream_valid(codec_dai, SNDRV_PCM_STREAM_PLAYBACK) &&
-			      snd_soc_dai_stream_valid(cpu_dai,   cpu_playback)))
+		/* Codec validation check */
+		for_each_rtd_codec_dais(rtd, i, dai) {
+			if (!snd_soc_dai_stream_valid(dai, SNDRV_PCM_STREAM_PLAYBACK))
 				has_playback = 0;
-			if (!(snd_soc_dai_stream_valid(codec_dai, SNDRV_PCM_STREAM_CAPTURE) &&
-			      snd_soc_dai_stream_valid(cpu_dai,   cpu_capture)))
+			if (!snd_soc_dai_stream_valid(dai, SNDRV_PCM_STREAM_CAPTURE))
 				has_capture = 0;
 		}
 	}