Re: [PATCH v2 2/3] remoteproc: qcom: pas: Map/unmap subsystem region before auth_and_reset

[kernel test robot <lkp@intel.com>]

Hi Mukesh,

kernel test robot noticed the following build warnings:

[auto build test WARNING on remoteproc/rproc-next]
[also build test WARNING on linus/master v7.0-rc5 next-20260327]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Mukesh-Ojha/remoteproc-qcom-pas-Map-unmap-subsystem-region-before-auth_and_reset/20260328-004624
base:   https://git.kernel.org/pub/scm/linux/kernel/git/remoteproc/linux.git rproc-next
patch link:    https://lore.kernel.org/r/20260325191301.164579-2-mukesh.ojha%40oss.qualcomm.com
patch subject: [PATCH v2 2/3] remoteproc: qcom: pas: Map/unmap subsystem region before auth_and_reset
config: arm64-randconfig-r113-20260328 (https://download.01.org/0day-ci/archive/20260328/202603281911.4W11adGI-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 8.5.0
sparse: v0.6.5-rc1
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260328/202603281911.4W11adGI-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603281911.4W11adGI-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/soc/qcom/mdt_loader.c:496:20: sparse: sparse: incorrect type in assignment (different address spaces) @@     expected void *mem_region @@     got void [noderef] __iomem * @@
   drivers/soc/qcom/mdt_loader.c:496:20: sparse:     expected void *mem_region
   drivers/soc/qcom/mdt_loader.c:496:20: sparse:     got void [noderef] __iomem *
>> drivers/soc/qcom/mdt_loader.c:505:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@     expected void volatile [noderef] __iomem *addr @@     got void *mem_region @@
   drivers/soc/qcom/mdt_loader.c:505:17: sparse:     expected void volatile [noderef] __iomem *addr
   drivers/soc/qcom/mdt_loader.c:505:17: sparse:     got void *mem_region

vim +496 drivers/soc/qcom/mdt_loader.c

   469	
   470	/**
   471	 * qcom_mdt_pas_load - Loads and authenticates the metadata of the firmware
   472	 * (typically contained in the .mdt file), followed by loading the actual
   473	 * firmware segments (e.g., .bXX files). Authentication of the segments done
   474	 * by a separate call.
   475	 *
   476	 * The PAS context must be initialized using qcom_scm_pas_context_init()
   477	 * prior to invoking this function.
   478	 *
   479	 * @ctx:        Pointer to the PAS (Peripheral Authentication Service) context
   480	 * @fw:         Firmware object representing the .mdt file
   481	 * @firmware:   Name of the firmware used to construct segment file names
   482	 * @reloc_base: Physical address adjusted after relocation
   483	 *
   484	 * Return: 0 on success or a negative error code on failure.
   485	 */
   486	int qcom_mdt_pas_load(struct qcom_scm_pas_context *ctx, const struct firmware *fw,
   487			      const char *firmware, phys_addr_t *reloc_base)
   488	{
   489		void *mem_region;
   490		int ret;
   491	
   492		ret = __qcom_mdt_pas_init(ctx->dev, fw, firmware, ctx->pas_id, ctx->mem_phys, ctx);
   493		if (ret)
   494			return ret;
   495	
 > 496		mem_region = ioremap_wc(ctx->mem_phys, ctx->mem_size);
   497		if (!mem_region) {
   498			dev_err(ctx->dev, "unable to map memory region: %pa+%zx\n", &ctx->mem_phys,
   499				ctx->mem_size);
   500			return -EINVAL;
   501		}
   502	
   503		ret = qcom_mdt_load_no_init(ctx->dev, fw, firmware, mem_region, ctx->mem_phys,
   504					    ctx->mem_size, reloc_base);
 > 505		iounmap(mem_region);
   506		return ret;
   507	}
   508	EXPORT_SYMBOL_GPL(qcom_mdt_pas_load);
   509	

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