diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-10-13 10:55:48 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2016-10-26 12:13:55 +0200 |
| commit | c8eabf821cac120afb78ca251b07cbf520406a7e (patch) | |
| tree | f8f337e7817ce1e12b0e9fb182a0ca096551622e /sound/soc/intel/skylake/skl-sst-utils.c | |
| parent | ASoC: intel: broadwell: constify snd_soc_ops structures (diff) | |
| download | linux-c8eabf821cac120afb78ca251b07cbf520406a7e.tar.xz linux-c8eabf821cac120afb78ca251b07cbf520406a7e.zip | |
ASoC: Intel: Skylake: Fix a shift wrapping bug
"*val" is a u64. It definitely looks like we intend to use the high 32
bits as well.
Fixes: 700a9a63f9c1 ("ASoC: Intel: Skylake: Add module instance id generation APIs")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Tested-by: Kranthi G <gudishax.kranthikumar@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/intel/skylake/skl-sst-utils.c')
| -rw-r--r-- | sound/soc/intel/skylake/skl-sst-utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/intel/skylake/skl-sst-utils.c b/sound/soc/intel/skylake/skl-sst-utils.c index 8dc03039b311..ea162fbf68e5 100644 --- a/sound/soc/intel/skylake/skl-sst-utils.c +++ b/sound/soc/intel/skylake/skl-sst-utils.c @@ -179,7 +179,7 @@ static inline int skl_getid_32(struct uuid_module *module, u64 *val, index = ffz(mask_val); pvt_id = index + word1_mask + word2_mask; if (pvt_id <= (max_inst - 1)) { - *val |= 1 << (index + word1_mask); + *val |= 1ULL << (index + word1_mask); return pvt_id; } } |