On Wed, 20 Nov 2024, Pei Xiao wrote: > sparse warning,expected 'void __iomem *addr',but got 'void *addr' Hi, Thanks for the patch. I've applied this patch to review-ilpo-next branch but I had to reformat the commit message. Please make sure you use proper spacing and put warnings like that on their own lines in future. -- i. > Reported-by: kernel test robot <lkp@xxxxxxxxx> > Closes: https://lore.kernel.org/oe-kbuild-all/202411121935.cgFcEMO4-lkp@xxxxxxxxx/ > Fixes: 423c3361855c ("platform/mellanox: mlxbf-pmc: Add support for BlueField-3") > Signed-off-by: Pei Xiao <xiaopei01@xxxxxxxxxx> > --- > drivers/platform/mellanox/mlxbf-pmc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/platform/mellanox/mlxbf-pmc.c b/drivers/platform/mellanox/mlxbf-pmc.c > index 9d18dfca6a67..9ff7b487dc48 100644 > --- a/drivers/platform/mellanox/mlxbf-pmc.c > +++ b/drivers/platform/mellanox/mlxbf-pmc.c > @@ -1168,7 +1168,7 @@ static int mlxbf_pmc_program_l3_counter(unsigned int blk_num, u32 cnt_num, u32 e > /* Method to handle crspace counter programming */ > static int mlxbf_pmc_program_crspace_counter(unsigned int blk_num, u32 cnt_num, u32 evt) > { > - void *addr; > + void __iomem *addr; > u32 word; > int ret; > > @@ -1192,7 +1192,7 @@ static int mlxbf_pmc_program_crspace_counter(unsigned int blk_num, u32 cnt_num, > /* Method to clear crspace counter value */ > static int mlxbf_pmc_clear_crspace_counter(unsigned int blk_num, u32 cnt_num) > { > - void *addr; > + void __iomem *addr; > > addr = pmc->block[blk_num].mmio_base + > MLXBF_PMC_CRSPACE_PERFMON_VAL0(pmc->block[blk_num].counters) + > @@ -1405,7 +1405,7 @@ static int mlxbf_pmc_read_l3_event(unsigned int blk_num, u32 cnt_num, u64 *resul > static int mlxbf_pmc_read_crspace_event(unsigned int blk_num, u32 cnt_num, u64 *result) > { > u32 word, evt; > - void *addr; > + void __iomem *addr; > int ret; > > addr = pmc->block[blk_num].mmio_base + >