diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-12-15 10:20:14 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-12-16 20:17:11 +0100 |
| commit | db4e561378b539230feb4db5e7d5d548c2db2cd4 (patch) | |
| tree | 99744d37a1c50068c7fa888fddd11901e6c77eae | |
| parent | ASoC: Intel: sst: fix sst_memcpy32 wrong with non-4x bytes issue (diff) | |
| download | linux-db4e561378b539230feb4db5e7d5d548c2db2cd4.tar.xz linux-db4e561378b539230feb4db5e7d5d548c2db2cd4.zip | |
ASoC: Intel: Skylake: Fix a couple signedness bugs
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>
| -rw-r--r-- | sound/soc/intel/skylake/skl-sst.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/soc/intel/skylake/skl-sst.c b/sound/soc/intel/skylake/skl-sst.c index 8cd5cdb21fd5..e26f4746afb7 100644 --- a/sound/soc/intel/skylake/skl-sst.c +++ b/sound/soc/intel/skylake/skl-sst.c @@ -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; |