ASoC: soc-dai: add snd_soc_pcm_dai_remove()
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Thu, 23 Apr 2020 23:15:20 +0000 (08:15 +0900)
committerMark Brown <broonie@kernel.org>
Wed, 29 Apr 2020 12:27:41 +0000 (13:27 +0100)
We have 2 type of component functions
snd_soc_dai_xxx()     is focusing to dai itself,
snd_soc_pcm_dai_xxx() is focusing to rtd related dai.

Now we can update snd_soc_dai_remove() to
snd_soc_pcm_dai_remove(). This patch do it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-By: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/87r1wdssjc.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
include/sound/soc-dai.h
sound/soc/soc-core.c
sound/soc/soc-dai.c

index bdb79df..cf7d09f 100644 (file)
@@ -158,12 +158,12 @@ snd_pcm_sframes_t snd_soc_dai_delay(struct snd_soc_dai *dai,
                                    struct snd_pcm_substream *substream);
 void snd_soc_dai_suspend(struct snd_soc_dai *dai);
 void snd_soc_dai_resume(struct snd_soc_dai *dai);
-int snd_soc_dai_remove(struct snd_soc_dai *dai);
 int snd_soc_dai_compress_new(struct snd_soc_dai *dai,
                             struct snd_soc_pcm_runtime *rtd, int num);
 bool snd_soc_dai_stream_valid(struct snd_soc_dai *dai, int stream);
 
 int snd_soc_pcm_dai_probe(struct snd_soc_pcm_runtime *rtd, int order);
+int snd_soc_pcm_dai_remove(struct snd_soc_pcm_runtime *rtd, int order);
 int snd_soc_pcm_dai_new(struct snd_soc_pcm_runtime *rtd);
 int snd_soc_pcm_dai_prepare(struct snd_pcm_substream *substream);
 int snd_soc_pcm_dai_trigger(struct snd_pcm_substream *substream, int cmd);
index 8cafca4..95d8189 100644 (file)
@@ -1256,35 +1256,15 @@ err_probe:
        return ret;
 }
 
-static void soc_remove_dai(struct snd_soc_dai *dai, int order)
-{
-       int err;
-
-       if (!dai || !dai->probed || !dai->driver ||
-           dai->driver->remove_order != order)
-               return;
-
-       err = snd_soc_dai_remove(dai);
-       if (err < 0)
-               dev_err(dai->dev,
-                       "ASoC: failed to remove %s: %d\n",
-                       dai->name, err);
-
-       dai->probed = 0;
-}
-
 static void soc_remove_link_dais(struct snd_soc_card *card)
 {
-       int i;
-       struct snd_soc_dai *dai;
        struct snd_soc_pcm_runtime *rtd;
        int order;
 
        for_each_comp_order(order) {
                for_each_card_rtds(card, rtd) {
-                       /* remove DAIs */
-                       for_each_rtd_dais(rtd, i, dai)
-                               soc_remove_dai(dai, order);
+                       /* remove all rtd connected DAIs in good order */
+                       snd_soc_pcm_dai_remove(rtd, order);
                }
        }
 }
index 48f5eb5..2bc452f 100644 (file)
@@ -366,16 +366,6 @@ snd_pcm_sframes_t snd_soc_dai_delay(struct snd_soc_dai *dai,
        return delay;
 }
 
-int snd_soc_dai_remove(struct snd_soc_dai *dai)
-{
-       int ret = 0;
-
-       if (dai->driver->remove)
-               ret = dai->driver->remove(dai);
-
-       return soc_dai_ret(dai, ret);
-}
-
 int snd_soc_dai_compress_new(struct snd_soc_dai *dai,
                             struct snd_soc_pcm_runtime *rtd, int num)
 {
@@ -420,6 +410,28 @@ int snd_soc_pcm_dai_probe(struct snd_soc_pcm_runtime *rtd, int order)
        return 0;
 }
 
+int snd_soc_pcm_dai_remove(struct snd_soc_pcm_runtime *rtd, int order)
+{
+       struct snd_soc_dai *dai;
+       int i, r, ret = 0;
+
+       for_each_rtd_dais(rtd, i, dai) {
+               if (dai->driver->remove_order != order)
+                       continue;
+
+               if (dai->probed &&
+                   dai->driver->remove) {
+                       r = dai->driver->remove(dai);
+                       if (r < 0)
+                               ret = r; /* use last error */
+               }
+
+               dai->probed = 0;
+       }
+
+       return ret;
+}
+
 int snd_soc_pcm_dai_new(struct snd_soc_pcm_runtime *rtd)
 {
        struct snd_soc_dai *dai;