diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-12-20 04:34:25 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-12-22 18:59:25 -0500 |
commit | bfbcab7c2d8ab4cb52f0785d7381d20f39bb065b (patch) | |
tree | a48356cfce1608b7ff424af655b58a746a684e5c | |
parent | 8005c49d9aea74d382f474ce11afbbc7d7130bec (diff) |
ASoC: ssm2518: Use a signed return type for ssm2518_lookup_mcs()
The return type "unsigned int" was used by the ssm2518_lookup_mcs()
function even though it will eventually return a negative error code.
Improve this implementation detail by deletion of the type modifier then.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/codecs/ssm2518.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/ssm2518.c b/sound/soc/codecs/ssm2518.c index 86b81a60ac52..e2e0bfa7ec20 100644 --- a/sound/soc/codecs/ssm2518.c +++ b/sound/soc/codecs/ssm2518.c | |||
@@ -309,7 +309,7 @@ static const struct snd_pcm_hw_constraint_list ssm2518_constraints_12288000 = { | |||
309 | .count = ARRAY_SIZE(ssm2518_rates_12288000), | 309 | .count = ARRAY_SIZE(ssm2518_rates_12288000), |
310 | }; | 310 | }; |
311 | 311 | ||
312 | static unsigned int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, | 312 | static int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, |
313 | unsigned int rate) | 313 | unsigned int rate) |
314 | { | 314 | { |
315 | const unsigned int *sysclks = NULL; | 315 | const unsigned int *sysclks = NULL; |