ASoC: wcd938x: simplify return value
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Wed, 4 Aug 2021 19:24:56 +0000 (14:24 -0500)
committerMark Brown <broonie@kernel.org>
Thu, 5 Aug 2021 14:38:54 +0000 (15:38 +0100)
cppcheck warning:

sound/soc/codecs/wcd938x.c:3701:9: warning: Identical condition and
return expression 'ret', return value is always 0
[identicalConditionAfterEarlyExit]

 return ret;
        ^
sound/soc/codecs/wcd938x.c:3691:6: note: If condition 'ret' is true,
        the function will return/exit
 if (ret)
     ^
sound/soc/codecs/wcd938x.c:3701:9: note: Returning identical expression 'ret'
 return ret;
        ^

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210804192456.278702-4-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wcd938x.c

index a627142..555904e 100644 (file)
@@ -3698,7 +3698,7 @@ static int wcd938x_probe(struct platform_device *pdev)
        pm_runtime_enable(dev);
        pm_runtime_idle(dev);
 
-       return ret;
+       return 0;
 }
 
 static int wcd938x_remove(struct platform_device *pdev)