diff options
| author | Pierre-Louis Bossart <[email protected]> | 2021-10-25 18:59:33 +0000 |
|---|---|---|
| committer | Mark Brown <[email protected]> | 2021-10-27 10:31:16 +0000 |
| commit | f913582190ddfe2380ecf8ee87b4ff2c8dcb5d48 (patch) | |
| tree | fdc3e529eb0a73db7cc465a257c717e6b6e2286a /lib/string_helpers.c | |
| parent | ASoC: mediatek: mt8195: fix return value (diff) | |
| download | kernel-f913582190ddfe2380ecf8ee87b4ff2c8dcb5d48.tar.gz kernel-f913582190ddfe2380ecf8ee87b4ff2c8dcb5d48.zip | |
ASoC: rockchip: i2s_tdm: improve return value handling
cppcheck reports the following warning:
sound/soc/rockchip/rockchip_i2s_tdm.c:599:9: warning: Identical
condition and return expression 'ret', return value is always 0
[identicalConditionAfterEarlyExit]
return ret;
^
sound/soc/rockchip/rockchip_i2s_tdm.c:594:6: note: If condition 'ret'
is true, the function will return/exit
if (ret)
^
sound/soc/rockchip/rockchip_i2s_tdm.c:599:9: note: Returning identical
expression 'ret'
return ret;
^
While the code is not wrong, it's clearer to return 0 directly.
Signed-off-by: Pierre-Louis Bossart <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions
