diff options
| author | Sang-Heon Jeon <ekffu200098@gmail.com> | 2026-07-26 01:03:41 +0900 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-07-27 14:07:58 +0100 |
| commit | 52c09577296275d96233429d6375b50fae24cd9c (patch) | |
| tree | 5098b13e7d43667ced7e8feb46a8cd24ca72bd9b | |
| parent | 3799a56da8f3250e5765344aa5d280678dc89124 (diff) | |
| download | linux-next-52c09577296275d96233429d6375b50fae24cd9c.tar.gz linux-next-52c09577296275d96233429d6375b50fae24cd9c.zip | |
ASoC: Intel: atom: 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: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://patch.msgid.link/20260725160344.916838-3-ekffu200098@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/intel/atom/sst-mfld-platform-pcm.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/sound/soc/intel/atom/sst-mfld-platform-pcm.c b/sound/soc/intel/atom/sst-mfld-platform-pcm.c index 9ee4d9926e06..a4c8cbfba096 100644 --- a/sound/soc/intel/atom/sst-mfld-platform-pcm.c +++ b/sound/soc/intel/atom/sst-mfld-platform-pcm.c @@ -236,12 +236,7 @@ static int sst_platform_alloc_stream(struct snd_pcm_substream *substream, stream->stream_info.str_id = str_params.stream_id; - ret_val = stream->ops->open(sst->dev, &str_params); - if (ret_val <= 0) - return ret_val; - - - return ret_val; + return stream->ops->open(sst->dev, &str_params); } static void sst_period_elapsed(void *arg) |