lkml.org 
[lkml]   [2016]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] clk: xgene: Don't call __pa on ioremaped address
Hi Laura,

[auto build test WARNING on clk/clk-next]
[also build test WARNING on v4.9-rc2 next-20161027]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url: https://github.com/0day-ci/linux/commits/Laura-Abbott/clk-xgene-Don-t-call-__pa-on-ioremaped-address/20161028-072455
base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

In file included from include/linux/kernel.h:13:0,
from include/linux/list.h:8,
from include/linux/module.h:9,
from drivers/clk/clk-xgene.c:23:
drivers/clk/clk-xgene.c: In function 'xgene_clk_enable':
>> drivers/clk/clk-xgene.c:489:12: warning: format '%p' expects argument of type 'void *', but argument 4 has type 'u32 {aka unsigned int}' [-Wformat=]
pr_debug("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n",
^
include/linux/printk.h:261:21: note: in definition of macro 'pr_fmt'
#define pr_fmt(fmt) fmt
^~~
include/linux/printk.h:309:2: note: in expansion of macro 'dynamic_pr_debug'
dynamic_pr_debug(fmt, ##__VA_ARGS__)
^~~~~~~~~~~~~~~~
>> drivers/clk/clk-xgene.c:489:3: note: in expansion of macro 'pr_debug'
pr_debug("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n",
^~~~~~~~
>> drivers/clk/clk-xgene.c:489:12: warning: format '%X' expects a matching 'unsigned int' argument [-Wformat=]
pr_debug("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n",
^
include/linux/printk.h:261:21: note: in definition of macro 'pr_fmt'
#define pr_fmt(fmt) fmt
^~~
include/linux/printk.h:309:2: note: in expansion of macro 'dynamic_pr_debug'
dynamic_pr_debug(fmt, ##__VA_ARGS__)
^~~~~~~~~~~~~~~~
>> drivers/clk/clk-xgene.c:489:3: note: in expansion of macro 'pr_debug'
pr_debug("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n",
^~~~~~~~

vim +489 drivers/clk/clk-xgene.c

308964ca Loc Ho 2013-06-26 473 data = xgene_clk_read(pclk->param.csr_reg +
308964ca Loc Ho 2013-06-26 474 pclk->param.reg_clk_offset);
308964ca Loc Ho 2013-06-26 475 data |= pclk->param.reg_clk_mask;
308964ca Loc Ho 2013-06-26 476 xgene_clk_write(data, pclk->param.csr_reg +
308964ca Loc Ho 2013-06-26 477 pclk->param.reg_clk_offset);
1ee1b519 Laura Abbott 2016-10-27 478 pr_debug("%s clk offset 0x%08X mask 0x%08X value 0x%08X\n",
1ee1b519 Laura Abbott 2016-10-27 479 clk_hw_get_name(hw),
308964ca Loc Ho 2013-06-26 480 pclk->param.reg_clk_offset, pclk->param.reg_clk_mask,
308964ca Loc Ho 2013-06-26 481 data);
308964ca Loc Ho 2013-06-26 482
308964ca Loc Ho 2013-06-26 483 /* Second enable the CSR */
308964ca Loc Ho 2013-06-26 484 data = xgene_clk_read(pclk->param.csr_reg +
308964ca Loc Ho 2013-06-26 485 pclk->param.reg_csr_offset);
308964ca Loc Ho 2013-06-26 486 data &= ~pclk->param.reg_csr_mask;
308964ca Loc Ho 2013-06-26 487 xgene_clk_write(data, pclk->param.csr_reg +
308964ca Loc Ho 2013-06-26 488 pclk->param.reg_csr_offset);
6ae5fd38 Stephen Boyd 2015-05-01 @489 pr_debug("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n",
1ee1b519 Laura Abbott 2016-10-27 490 clk_hw_get_name(hw),
308964ca Loc Ho 2013-06-26 491 pclk->param.reg_csr_offset, pclk->param.reg_csr_mask,
308964ca Loc Ho 2013-06-26 492 data);
308964ca Loc Ho 2013-06-26 493 }
308964ca Loc Ho 2013-06-26 494
308964ca Loc Ho 2013-06-26 495 if (pclk->lock)
308964ca Loc Ho 2013-06-26 496 spin_unlock_irqrestore(pclk->lock, flags);
308964ca Loc Ho 2013-06-26 497

:::::: The code at line 489 was first introduced by commit
:::::: 6ae5fd381251993a650a7fad51bd9318c19a6c80 clk: xgene: Silence sparse warnings

:::::: TO: Stephen Boyd <sboyd@codeaurora.org>
:::::: CC: Stephen Boyd <sboyd@codeaurora.org>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2016-10-28 02:34    [W:0.048 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site