* Re: [PATCH v4 2/4] misc: fastrpc: Rename phys to dma_addr for clarity
[not found] <20251126094545.2139376-3-kumari.pallavi@oss.qualcomm.com>
@ 2025-11-27 12:23 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2025-11-27 12:23 UTC (permalink / raw)
To: Kumari Pallavi, kpallavi, srini, amahesh, arnd, gregkh, robh,
krzk+dt, conor+dt
Cc: llvm, oe-kbuild-all, Kumari Pallavi, quic_bkumar, ekansh.gupta,
linux-kernel, quic_chennak, dri-devel, linux-arm-msm, devicetree,
jingyi.wang, aiqun.yu, ktadakam
Hi Kumari,
kernel test robot noticed the following build warnings:
[auto build test WARNING on char-misc/char-misc-testing]
[also build test WARNING on char-misc/char-misc-next char-misc/char-misc-linus robh/for-next soc/for-next linus/master v6.18-rc7 next-20251127]
[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/Kumari-Pallavi/dt-bindings-misc-qcom-fastrpc-Add-compatible-for-Kaanapali/20251126-175106
base: char-misc/char-misc-testing
patch link: https://lore.kernel.org/r/20251126094545.2139376-3-kumari.pallavi%40oss.qualcomm.com
patch subject: [PATCH v4 2/4] misc: fastrpc: Rename phys to dma_addr for clarity
config: arm-randconfig-002-20251127 (https://download.01.org/0day-ci/archive/20251127/202511272058.teHG4sdy-lkp@intel.com/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 9e9fe08b16ea2c4d9867fb4974edf2a3776d6ece)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251127/202511272058.teHG4sdy-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/202511272058.teHG4sdy-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/misc/fastrpc.c:328:6: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
327 | "Failed to assign memory dma_addr 0x%llx size 0x%llx err %d\n",
| ~~~~
| %x
328 | map->dma_addr, map->len, err);
| ^~~~~~~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
drivers/misc/fastrpc.c:823:5: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
822 | "Failed to assign memory with dma_addr 0x%llx size 0x%llx err %d\n",
| ~~~~
| %x
823 | map->dma_addr, map->len, err);
| ^~~~~~~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
drivers/misc/fastrpc.c:1318:6: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
1317 | "Failed to assign memory with dma_addr 0x%llx size 0x%llx err %d\n",
| ~~~~
| %x
1318 | fl->cctx->remote_heap->dma_addr,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
drivers/misc/fastrpc.c:1373:5: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
1372 | dev_err(fl->sctx->dev, "Failed to assign memory dma_addr 0x%llx size 0x%llx err %d\n",
| ~~~~
| %x
1373 | fl->cctx->remote_heap->dma_addr, fl->cctx->remote_heap->size, err);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
drivers/misc/fastrpc.c:1953:5: warning: format specifies type 'unsigned long long' but the argument has type 'dma_addr_t' (aka 'unsigned int') [-Wformat]
1952 | "Failed to assign memory dma_addr 0x%llx size 0x%llx err %d",
| ~~~~
| %x
1953 | buf->dma_addr, buf->size, err);
| ^~~~~~~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
5 warnings generated.
vim +328 drivers/misc/fastrpc.c
307
308 static void fastrpc_free_map(struct kref *ref)
309 {
310 struct fastrpc_map *map;
311
312 map = container_of(ref, struct fastrpc_map, refcount);
313
314 if (map->table) {
315 if (map->attr & FASTRPC_ATTR_SECUREMAP) {
316 struct qcom_scm_vmperm perm;
317 int vmid = map->fl->cctx->vmperms[0].vmid;
318 u64 src_perms = BIT(QCOM_SCM_VMID_HLOS) | BIT(vmid);
319 int err = 0;
320
321 perm.vmid = QCOM_SCM_VMID_HLOS;
322 perm.perm = QCOM_SCM_PERM_RWX;
323 err = qcom_scm_assign_mem(map->dma_addr, map->len,
324 &src_perms, &perm, 1);
325 if (err) {
326 dev_err(map->fl->sctx->dev,
327 "Failed to assign memory dma_addr 0x%llx size 0x%llx err %d\n",
> 328 map->dma_addr, map->len, err);
329 return;
330 }
331 }
332 dma_buf_unmap_attachment_unlocked(map->attach, map->table,
333 DMA_BIDIRECTIONAL);
334 dma_buf_detach(map->buf, map->attach);
335 dma_buf_put(map->buf);
336 }
337
338 if (map->fl) {
339 spin_lock(&map->fl->lock);
340 list_del(&map->node);
341 spin_unlock(&map->fl->lock);
342 map->fl = NULL;
343 }
344
345 kfree(map);
346 }
347
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread