diff options
| author | Sang-Heon Jeon <ekffu200098@gmail.com> | 2026-07-26 01:03:40 +0900 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-07-27 14:07:57 +0100 |
| commit | 3799a56da8f3250e5765344aa5d280678dc89124 (patch) | |
| tree | 57e7e9eb71aa455253a16c3ab985a132b3ba6e08 | |
| parent | 7859b74c0bd600cddd86afd55a3dde285a8c2937 (diff) | |
| download | linux-next-3799a56da8f3250e5765344aa5d280678dc89124.tar.gz linux-next-3799a56da8f3250e5765344aa5d280678dc89124.zip | |
ASoC: amd: acp: remove conditional return with no effect
Both branches of the check return the same value, so the check has
no effect. Remove it and return the value directly.
This is the result of running the Coccinelle script from
scripts/coccinelle/misc/cond_return_no_effect.cocci.
Signed-off-by: Sang-Heon Jeon <ekffu200098@gmail.com>
Reviewed-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
Link: https://patch.msgid.link/20260725160344.916838-2-ekffu200098@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/amd/acp/acp-mach-common.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/sound/soc/amd/acp/acp-mach-common.c b/sound/soc/amd/acp/acp-mach-common.c index ef784cca13f2..01a0aaa60246 100644 --- a/sound/soc/amd/acp/acp-mach-common.c +++ b/sound/soc/amd/acp/acp-mach-common.c @@ -938,15 +938,10 @@ static int acp_max98388_hw_params(struct snd_pcm_substream *substream, struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, MAX98388_CODEC_DAI); - int ret; - ret = snd_soc_dai_set_fmt(codec_dai, - SND_SOC_DAIFMT_CBC_CFC | SND_SOC_DAIFMT_I2S | - SND_SOC_DAIFMT_NB_NF); - if (ret < 0) - return ret; - - return ret; + return snd_soc_dai_set_fmt(codec_dai, + SND_SOC_DAIFMT_CBC_CFC | SND_SOC_DAIFMT_I2S | + SND_SOC_DAIFMT_NB_NF); } static const struct snd_soc_ops acp_max98388_ops = { |