ASoC: cs42l43: Remove useless else
authorJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Thu, 28 Sep 2023 08:52:00 +0000 (16:52 +0800)
committerMark Brown <broonie@kernel.org>
Thu, 28 Sep 2023 09:53:21 +0000 (11:53 +0200)
The assignment of the else and if branches is the same, so the else
here is redundant, so we remove it.

./sound/soc/codecs/cs42l43-sdw.c:35:1-3: WARNING: possible condition with no effect (if == else).

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=6712
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230928085200.48635-1-jiapeng.chong@linux.alibaba.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/cs42l43-sdw.c

index 55ac5fe8c3db86dee31c8dafbbf6577e850041a6..388f95853b6998c163905e58fa01a79896a971ca 100644 (file)
@@ -31,11 +31,7 @@ int cs42l43_sdw_add_peripheral(struct snd_pcm_substream *substream,
                return -EINVAL;
 
        snd_sdw_params_to_config(substream, params, &sconfig, &pconfig);
-
-       if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK)
-               pconfig.num = dai->id;
-       else
-               pconfig.num = dai->id;
+       pconfig.num = dai->id;
 
        ret = sdw_stream_add_slave(sdw, &sconfig, &pconfig, 1, sdw_stream);
        if (ret) {