diff options
author | Dan Carpenter <[email protected]> | 2016-10-13 11:55:48 +0300 |
---|---|---|
committer | Mark Brown <[email protected]> | 2016-10-26 11:13:55 +0100 |
commit | c8eabf821cac120afb78ca251b07cbf520406a7e (patch) | |
tree | f8f337e7817ce1e12b0e9fb182a0ca096551622e | |
parent | d5bc18c148fd3d3ab7cbd3d976f7e27226f74dd8 (diff) |
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 <[email protected]>
Acked-by: Vinod Koul <[email protected]>
Tested-by: Kranthi G <[email protected]>
Signed-off-by: Mark Brown <[email protected]>
-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; } } |