diff options
author | 2021-03-15 11:39:30 -0500 | |
---|---|---|
committer | 2021-03-16 13:25:44 +0000 | |
commit | 11fd6d8e097b5b179ea445e0206aaefc47e62845 (patch) | |
tree | fd6437a28c608273fc4ebdb58eb80653e67fda3e /sound/soc/sof/core.c | |
parent | Merge series "AsoC: rt5640/rt5651: Volume control fixes" from Hans de Goede <... (diff) | |
download | linux-11fd6d8e097b5b179ea445e0206aaefc47e62845.tar.gz linux-11fd6d8e097b5b179ea445e0206aaefc47e62845.tar.bz2 linux-11fd6d8e097b5b179ea445e0206aaefc47e62845.zip |
ASoC: SOF: fix debugfs initialisation error handling
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>
Diffstat (limited to 'sound/soc/sof/core.c')
-rw-r--r-- | sound/soc/sof/core.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sound/soc/sof/core.c b/sound/soc/sof/core.c index 6d8f7d9fd192..495295a34c3a 100644 --- a/sound/soc/sof/core.c +++ b/sound/soc/sof/core.c @@ -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 */ |