diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-02-14 11:10:36 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2018-02-15 10:14:48 -0500 |
commit | 53b8e4504aaa7fc88fd18e6caac3a4d4367bd4a4 (patch) | |
tree | f2f29598377cee18a56e933a35b087b31ff6e865 | |
parent | 21957b5bf0ff607e6e0ed55a3a89454e969511ba (diff) |
ASoC: uniphier: remove redundant check of blk_id
The check of blk_id == AUD_CLK_IO is redundant as it also being performed
in the following switch statement with the same return of -ENOTSUPP. Fix
this by removing the redundant comparison.
Detected by CoverityScan, CID#1465227 ("Logically dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/uniphier/aio-cpu.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sound/soc/uniphier/aio-cpu.c b/sound/soc/uniphier/aio-cpu.c index 55f3248a31fd..7cf2316c69a2 100644 --- a/sound/soc/uniphier/aio-cpu.c +++ b/sound/soc/uniphier/aio-cpu.c | |||
@@ -134,9 +134,6 @@ static int uniphier_aio_set_sysclk(struct snd_soc_dai *dai, int clk_id, | |||
134 | bool pll_auto = false; | 134 | bool pll_auto = false; |
135 | int pll_id, div_id; | 135 | int pll_id, div_id; |
136 | 136 | ||
137 | if (clk_id == AUD_CLK_IO) | ||
138 | return -ENOTSUPP; | ||
139 | |||
140 | switch (clk_id) { | 137 | switch (clk_id) { |
141 | case AUD_CLK_IO: | 138 | case AUD_CLK_IO: |
142 | return -ENOTSUPP; | 139 | return -ENOTSUPP; |