ALSA: hda/cs35l56: Do some clean up on probe error
authorDan Carpenter <dan.carpenter@linaro.org>
Thu, 27 Jul 2023 07:16:33 +0000 (10:16 +0300)
committerTakashi Iwai <tiwai@suse.de>
Thu, 27 Jul 2023 09:53:45 +0000 (11:53 +0200)
Smatch complains that this return should be a goto:

    sound/pci/hda/cs35l56_hda.c:910 cs35l56_hda_common_probe()
    warn: missing unwind goto?

The goto error disables cansleep so that seems reasonable.

Fixes: 73cfbfa9caea ("ALSA: hda/cs35l56: Add driver for Cirrus Logic CS35L56 amplifier")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/465160f4-b7cf-41d5-931e-d6c9e68fa3c7@moroto.mountain
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/cs35l56_hda.c

index 71e95e6..4c3279f 100644 (file)
@@ -907,7 +907,7 @@ int cs35l56_hda_common_probe(struct cs35l56_hda *cs35l56, int id)
 
        ret = cs35l56_set_patch(&cs35l56->base);
        if (ret)
-               return ret;
+               goto err;
 
        regcache_mark_dirty(cs35l56->base.regmap);
        regcache_sync(cs35l56->base.regmap);