diff options
author | Ying Liu <lyre@hust.edu.cn> | 2023-04-11 19:09:12 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2023-04-12 13:52:00 +0200 |
commit | d93ee84e3eb5d1afc081e57ca37f1411a01f2c94 (patch) | |
tree | 5ab297fb1120df4df2e452b77139c089c011280e /sound/soc | |
parent | ASoC: SOF: Intel: mtl: fix page fault in dspless mode when DSP is disabled (diff) | |
download | linux-d93ee84e3eb5d1afc081e57ca37f1411a01f2c94.tar.xz linux-d93ee84e3eb5d1afc081e57ca37f1411a01f2c94.zip |
ASoC: tas5720: add missing unwind goto in tas5720_codec_probe
Smatch complains that missing unwind goto in tas5720_codec_probe.
When tas5720 has an invalid devtype, it is expected to invoke
regulator_bulk_disable to handle the failure. But the default
option return an error code directly. Fix it by reusing the
probe_fail label.
Signed-off-by: Ying Liu <lyre@hust.edu.cn>
Link: https://lore.kernel.org/r/20230411170912.1939906-1-lyre@hust.edu.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc')
-rw-r--r-- | sound/soc/codecs/tas5720.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/soc/codecs/tas5720.c b/sound/soc/codecs/tas5720.c index de6d01c8fdd3..4d27b60bd804 100644 --- a/sound/soc/codecs/tas5720.c +++ b/sound/soc/codecs/tas5720.c @@ -339,7 +339,8 @@ static int tas5720_codec_probe(struct snd_soc_component *component) break; default: dev_err(component->dev, "unexpected private driver data\n"); - return -EINVAL; + ret = -EINVAL; + goto probe_fail; } if (device_id != expected_device_id) |