ASoC: SOF: fix debugfs initialisation error handling
authorGuennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Mon, 15 Mar 2021 16:39:30 +0000 (11:39 -0500)
committerMark Brown <broonie@kernel.org>
Tue, 16 Mar 2021 13:25:44 +0000 (13:25 +0000)
If debugfs initialisation fails partially in sof_probe_continue() some
debugfs files and the root directory might have been created
successfully. They have to be cleaned up if some of them failed too.

Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Xiuli Pan <xiulipan@outlook.com>
Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210315163932.18663-2-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/core.c

index 6d8f7d9fd192057ac5b25f245a7c405d30d73928..495295a34c3a692cb340f8b5650fe8684b7cee26 100644 (file)
@@ -154,7 +154,7 @@ static int sof_probe_continue(struct snd_sof_dev *sdev)
        if (ret < 0) {
                dev_err(sdev->dev, "error: failed to get machine info %d\n",
                        ret);
-               goto dbg_err;
+               goto dsp_err;
        }
 
        /* set up platform component driver */
@@ -257,8 +257,9 @@ fw_run_err:
 fw_load_err:
        snd_sof_ipc_free(sdev);
 ipc_err:
-       snd_sof_free_debug(sdev);
 dbg_err:
+       snd_sof_free_debug(sdev);
+dsp_err:
        snd_sof_remove(sdev);
 
        /* all resources freed, update state to match */