ASoC: codecs: tas2780: remove redundant assignments to variable ret
authorColin Ian King <colin.i.king@gmail.com>
Thu, 11 Apr 2024 08:33:32 +0000 (09:33 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 11 Apr 2024 12:30:12 +0000 (13:30 +0100)
Variable ret is being assigned a value that is never read in a couple
of places. The variable is being re-assigned later on. The assignments
are redundant and can be removed.

Cleans up clang scan build warning:
sound/soc/codecs/tas2780.c:84:2: warning: Value stored to 'ret' is
never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://msgid.link/r/20240411083332.304887-1-colin.i.king@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/tas2780.c

index 41076be238542bbe4ee94a44acff6c2ece4e8d31..972e8ea5ebde7e343118182c1ceb5ad9d2d4bd96 100644 (file)
@@ -71,7 +71,7 @@ static int tas2780_codec_resume(struct snd_soc_component *component)
 {
        struct tas2780_priv *tas2780 =
                snd_soc_component_get_drvdata(component);
-       int ret = 0;
+       int ret;
 
        ret = snd_soc_component_update_bits(component, TAS2780_PWR_CTRL,
                TAS2780_PWR_CTRL_MASK, TAS2780_PWR_CTRL_ACTIVE);
@@ -81,7 +81,6 @@ static int tas2780_codec_resume(struct snd_soc_component *component)
                        __func__, ret);
                goto err;
        }
-       ret = 0;
        regcache_cache_only(tas2780->regmap, false);
        ret = regcache_sync(tas2780->regmap);
 err: