ASoC: topology: change the complete op in snd_soc_tplg_ops to return int

In the SOF driver, the operations performed in the complete callback
can fail and therefore topology loading should return an error in
such cases. So, change the signature of the complete op
in struct snd_soc_tplg_ops to return an int to return the error.

Also, amend the complete callback functions in the SOF driver and
the SKL driver to conform with the new signature.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Link: https://lore.kernel.org/r/20210927120517.20505-2-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/include/sound/soc-topology.h b/include/sound/soc-topology.h
index 4afd667e124c23a3a07fd5932b7416fe0841ae94..7f33de8ffd95a39cdf7821f31bc7c208d8324392 100644 (file)
--- a/include/sound/soc-topology.h
+++ b/include/sound/soc-topology.h
@@ -151,7 +151,7 @@ struct snd_soc_tplg_ops {
                struct snd_soc_tplg_hdr *);
 
        /* completion - called at completion of firmware loading */
-       void (*complete)(struct snd_soc_component *);
+       int (*complete)(struct snd_soc_component *comp);
 
        /* manifest - optional to inform component of manifest */
        int (*manifest)(struct snd_soc_component *, int index,

diff --git a/sound/soc/intel/skylake/skl-topology.c b/sound/soc/intel/skylake/skl-topology.c
index b036852d68893eaf74e847abcdc6abc693c597dd..89e4231304dda0f4e30bc5641006bba920fc504e 100644 (file)
--- a/sound/soc/intel/skylake/skl-topology.c
+++ b/sound/soc/intel/skylake/skl-topology.c
@@ -3637,7 +3637,7 @@ static int skl_manifest_load(struct snd_soc_component *cmpnt, int index,
        return 0;
 }
 
-static void skl_tplg_complete(struct snd_soc_component *component)
+static int skl_tplg_complete(struct snd_soc_component *component)
 {
        struct snd_soc_dobj *dobj;
        struct snd_soc_acpi_mach *mach;
@@ -3646,7 +3646,7 @@ static void skl_tplg_complete(struct snd_soc_component *component)
 
        val = kmalloc(sizeof(*val), GFP_KERNEL);
        if (!val)
-               return;
+               return -ENOMEM;
 
        mach = dev_get_platdata(component->card->dev);
        list_for_each_entry(dobj, &component->dobj_list, list) {
@@ -3671,7 +3671,9 @@ static void skl_tplg_complete(struct snd_soc_component *component)
                        }
                }
        }
+
        kfree(val);
+       return 0;
 }
 
 static struct snd_soc_tplg_ops skl_tplg_ops  = {

diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c
index 73e1b7b48ce98ee85bbef2e3bfd6b16d3f1d73a4..88f849b119da6aa8f638581dc9997643fe9103b5 100644 (file)
--- a/sound/soc/soc-topology.c
+++ b/sound/soc/soc-topology.c
@@ -78,7 +78,7 @@ struct soc_tplg {
 };
 
 static int soc_tplg_process_headers(struct soc_tplg *tplg);
-static void soc_tplg_complete(struct soc_tplg *tplg);
+static int soc_tplg_complete(struct soc_tplg *tplg);
 
 /* check we dont overflow the data for this control chunk */
 static int soc_tplg_check_elem_count(struct soc_tplg *tplg, size_t elem_size,
@@ -312,10 +312,12 @@ static int soc_tplg_dai_link_load(struct soc_tplg *tplg,
 }
 
 /* tell the component driver that all firmware has been loaded in this request */
-static void soc_tplg_complete(struct soc_tplg *tplg)
+static int soc_tplg_complete(struct soc_tplg *tplg)
 {
        if (tplg->ops && tplg->ops->complete)
-               tplg->ops->complete(tplg->comp);
+               return tplg->ops->complete(tplg->comp);
+
+       return 0;
 }
 
 /* add a dynamic kcontrol */
@@ -2625,7 +2627,7 @@ static int soc_tplg_load(struct soc_tplg *tplg)
 
        ret = soc_tplg_process_headers(tplg);
        if (ret == 0)
-               soc_tplg_complete(tplg);
+               return soc_tplg_complete(tplg);
 
        return ret;
 }

diff --git a/sound/soc/sof/topology.c b/sound/soc/sof/topology.c
index cc9585bfa4e9f24f801615b23e24a8e1e83529e2..96b8791f7cc1eebf705c88ffd629340e63ae3569 100644 (file)
--- a/sound/soc/sof/topology.c
+++ b/sound/soc/sof/topology.c
@@ -3567,10 +3567,11 @@ int snd_sof_complete_pipeline(struct device *dev,
 }
 
 /* completion - called at completion of firmware loading */
-static void sof_complete(struct snd_soc_component *scomp)
+static int sof_complete(struct snd_soc_component *scomp)
 {
        struct snd_sof_dev *sdev = snd_soc_component_get_drvdata(scomp);
        struct snd_sof_widget *swidget;
+       int ret;
 
        /* some widget types require completion notificattion */
        list_for_each_entry(swidget, &sdev->widget_list, list) {
@@ -3579,8 +3580,11 @@ static void sof_complete(struct snd_soc_component *scomp)
 
                switch (swidget->id) {
                case snd_soc_dapm_scheduler:
-                       swidget->complete =
-                               snd_sof_complete_pipeline(scomp->dev, swidget);
+                       ret = snd_sof_complete_pipeline(scomp->dev, swidget);
+                       if (ret < 0)
+                               return ret;
+
+                       swidget->complete = ret;
                        break;
                default:
                        break;
@@ -3590,7 +3594,7 @@ static void sof_complete(struct snd_soc_component *scomp)
         * cache initial values of SOF kcontrols by reading DSP value over
         * IPC. It may be overwritten by alsa-mixer after booting up
         */
-       snd_sof_cache_kcontrol_val(scomp);
+       return snd_sof_cache_kcontrol_val(scomp);
 }
 
 /* manifest - optional to inform component of manifest */