diff options
author | 2012-12-20 16:53:16 +0800 | |
---|---|---|
committer | 2012-12-24 16:02:19 +0000 | |
commit | 3271a4fc7daeb489bfe1730023c166065e6fb0e7 (patch) | |
tree | 13d8c60aae948308e13cf05b84a8dd526897c50f /sound | |
parent | Linux 3.8-rc1 (diff) | |
download | linux-3271a4fc7daeb489bfe1730023c166065e6fb0e7.tar.gz linux-3271a4fc7daeb489bfe1730023c166065e6fb0e7.tar.bz2 linux-3271a4fc7daeb489bfe1730023c166065e6fb0e7.zip |
ASoC: cs42l52: Catch no-match case in cs42l52_get_clk
In the case of no-match, return -EINVAL instead of 0.
Since we assign i to ret in the for loop, ret always less than
ARRAY_SIZE(clk_map_table). Thus remove the boundary checking for ret.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/soc/codecs/cs42l52.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sound/soc/codecs/cs42l52.c b/sound/soc/codecs/cs42l52.c index 99bb1c69499e..9811a5478c87 100644 --- a/sound/soc/codecs/cs42l52.c +++ b/sound/soc/codecs/cs42l52.c @@ -737,7 +737,7 @@ static const struct cs42l52_clk_para clk_map_table[] = { static int cs42l52_get_clk(int mclk, int rate) { - int i, ret = 0; + int i, ret = -EINVAL; u_int mclk1, mclk2 = 0; for (i = 0; i < ARRAY_SIZE(clk_map_table); i++) { @@ -749,8 +749,6 @@ static int cs42l52_get_clk(int mclk, int rate) } } } - if (ret > ARRAY_SIZE(clk_map_table)) - return -EINVAL; return ret; } |