perf/smmuv3: Don't cast parameter in bit operations
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 9 Feb 2022 18:47:58 +0000 (20:47 +0200)
committerWill Deacon <will@kernel.org>
Tue, 15 Feb 2022 16:51:26 +0000 (16:51 +0000)
commit8ddf4eff71e12e4295b54ac8b02439ad22271fd2
tree796e3ccd8c57fcfe6af7298f135031743832c1e3
parent30de2b541af98179780054836b48825fcfba4408
perf/smmuv3: Don't cast parameter in bit operations

While in this particular case it would not be a (critical) issue,
the pattern itself is bad and error prone in case somebody blindly
copies to their code.

Don't cast parameter to unsigned long pointer in the bit operations.
Instead copy to a local variable on stack of a proper type and use.

Note, new compilers might warn on this line for potential outbound access.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/20220209184758.56578-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Will Deacon <will@kernel.org>
drivers/perf/arm_smmuv3_pmu.c