ASoC: Intel: Skylake: Fix a couple signedness bugs
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 15 Dec 2015 09:20:14 +0000 (12:20 +0300)
committerMark Brown <broonie@kernel.org>
Wed, 16 Dec 2015 19:17:11 +0000 (19:17 +0000)
These need to be signed because they hold negative error codes.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/skylake/skl-sst.c

index 8cd5cdb21fd5d24de23656e466527c16bf2eeb7f..e26f4746afb7e58a306ea453da996c77571a5431 100644 (file)
@@ -208,7 +208,7 @@ static unsigned int skl_get_errorcode(struct sst_dsp *ctx)
  * since get/set_module are called from DAPM context,
  * we don't need lock for usage count
  */
-static unsigned int skl_get_module(struct sst_dsp *ctx, u16 mod_id)
+static int skl_get_module(struct sst_dsp *ctx, u16 mod_id)
 {
        struct skl_module_table *module;
 
@@ -220,7 +220,7 @@ static unsigned int skl_get_module(struct sst_dsp *ctx, u16 mod_id)
        return -EINVAL;
 }
 
-static unsigned int skl_put_module(struct sst_dsp *ctx, u16 mod_id)
+static int skl_put_module(struct sst_dsp *ctx, u16 mod_id)
 {
        struct skl_module_table *module;
 
@@ -340,7 +340,7 @@ static int skl_load_module(struct sst_dsp *ctx, u16 mod_id, char *guid)
 
 static int skl_unload_module(struct sst_dsp *ctx, u16 mod_id)
 {
-       unsigned int usage_cnt;
+       int usage_cnt;
        struct skl_sst *skl = ctx->thread_context;
        int ret = 0;