| Message ID | 20210804192456.278702-4-pierre-louis.bossart@linux.intel.com |
|---|---|
| State | Accepted |
| Commit | 8c62dbcb489aa038fcb9c73faa5c6b56f1f54902 |
| Headers | show |
| Series | [1/3] ASoC: max98090: remove duplicate status reads and useless assignmment | expand |
diff --git a/sound/soc/codecs/wcd938x.c b/sound/soc/codecs/wcd938x.c index dd69653ceec0..a1d03cfcf92d 100644 --- a/sound/soc/codecs/wcd938x.c +++ b/sound/soc/codecs/wcd938x.c @@ -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)
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> --- sound/soc/codecs/wcd938x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)