drivers/remoteproc/stm32_rproc.c:139:20: sparse: sparse: incorrect type in argument 1 (different address spaces)

From: kernel test robot
Date: Fri Nov 17 2023 - 22:58:36 EST


Hi Arnd,

First bad commit (maybe != root cause):

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 791c8ab095f71327899023223940dd52257a4173
commit: 03bd158e1535e68bcd2b1e095b0ebcad7c84bd20 remoteproc: stm32: use correct format strings on 64-bit
date: 5 months ago
config: arm-randconfig-r121-20231118 (https://download.01.org/0day-ci/archive/20231118/202311181131.FV6wkpIs-lkp@xxxxxxxxx/config)
compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
reproduce: (https://download.01.org/0day-ci/archive/20231118/202311181131.FV6wkpIs-lkp@xxxxxxxxx/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311181131.FV6wkpIs-lkp@xxxxxxxxx/

sparse warnings: (new ones prefixed by >>)
drivers/remoteproc/stm32_rproc.c:122:12: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected void *va @@ got void [noderef] __iomem * @@
drivers/remoteproc/stm32_rproc.c:122:12: sparse: expected void *va
drivers/remoteproc/stm32_rproc.c:122:12: sparse: got void [noderef] __iomem *
>> drivers/remoteproc/stm32_rproc.c:139:20: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void volatile [noderef] __iomem *io_addr @@ got void *va @@
drivers/remoteproc/stm32_rproc.c:139:20: sparse: expected void volatile [noderef] __iomem *io_addr
drivers/remoteproc/stm32_rproc.c:139:20: sparse: got void *va
drivers/remoteproc/stm32_rproc.c:644:17: sparse: sparse: cast removes address space '__iomem' of expression
drivers/remoteproc/stm32_rproc.c: note: in included file (through include/linux/mmzone.h, include/linux/gfp.h, include/linux/xarray.h, ...):
include/linux/page-flags.h:240:46: sparse: sparse: self-comparison always evaluates to false

vim +139 drivers/remoteproc/stm32_rproc.c

13140de09cc2dd Fabien Dessenne 2019-05-14 114
13140de09cc2dd Fabien Dessenne 2019-05-14 115 static int stm32_rproc_mem_alloc(struct rproc *rproc,
13140de09cc2dd Fabien Dessenne 2019-05-14 116 struct rproc_mem_entry *mem)
13140de09cc2dd Fabien Dessenne 2019-05-14 117 {
13140de09cc2dd Fabien Dessenne 2019-05-14 118 struct device *dev = rproc->dev.parent;
13140de09cc2dd Fabien Dessenne 2019-05-14 119 void *va;
13140de09cc2dd Fabien Dessenne 2019-05-14 120
03bd158e1535e6 Arnd Bergmann 2023-06-09 121 dev_dbg(dev, "map memory: %pad+%zx\n", &mem->dma, mem->len);
13140de09cc2dd Fabien Dessenne 2019-05-14 @122 va = ioremap_wc(mem->dma, mem->len);
13140de09cc2dd Fabien Dessenne 2019-05-14 123 if (IS_ERR_OR_NULL(va)) {
03bd158e1535e6 Arnd Bergmann 2023-06-09 124 dev_err(dev, "Unable to map memory region: %pad+0x%zx\n",
13140de09cc2dd Fabien Dessenne 2019-05-14 125 &mem->dma, mem->len);
13140de09cc2dd Fabien Dessenne 2019-05-14 126 return -ENOMEM;
13140de09cc2dd Fabien Dessenne 2019-05-14 127 }
13140de09cc2dd Fabien Dessenne 2019-05-14 128
13140de09cc2dd Fabien Dessenne 2019-05-14 129 /* Update memory entry va */
13140de09cc2dd Fabien Dessenne 2019-05-14 130 mem->va = va;
13140de09cc2dd Fabien Dessenne 2019-05-14 131
13140de09cc2dd Fabien Dessenne 2019-05-14 132 return 0;
13140de09cc2dd Fabien Dessenne 2019-05-14 133 }
13140de09cc2dd Fabien Dessenne 2019-05-14 134
13140de09cc2dd Fabien Dessenne 2019-05-14 135 static int stm32_rproc_mem_release(struct rproc *rproc,
13140de09cc2dd Fabien Dessenne 2019-05-14 136 struct rproc_mem_entry *mem)
13140de09cc2dd Fabien Dessenne 2019-05-14 137 {
13140de09cc2dd Fabien Dessenne 2019-05-14 138 dev_dbg(rproc->dev.parent, "unmap memory: %pa\n", &mem->dma);
13140de09cc2dd Fabien Dessenne 2019-05-14 @139 iounmap(mem->va);
13140de09cc2dd Fabien Dessenne 2019-05-14 140
13140de09cc2dd Fabien Dessenne 2019-05-14 141 return 0;
13140de09cc2dd Fabien Dessenne 2019-05-14 142 }
13140de09cc2dd Fabien Dessenne 2019-05-14 143

:::::: The code at line 139 was first introduced by commit
:::::: 13140de09cc2dd5e5166ad42292bb82af4e23cef remoteproc: stm32: add an ST stm32_rproc driver

:::::: TO: Fabien Dessenne <fabien.dessenne@xxxxxx>
:::::: CC: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki