[andersson-kernel:wip/for-robert/dsi-panel-fixup 8155/8157] drivers/clk/qcom/clk-rcg2.c:109:16: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'long unsigned int'

[kernel test robot]

tree:   https://github.com/andersson/kernel wip/for-robert/dsi-panel-fixup
head:   2c9fc6fc5e7be5c2b042e98045f2b0c763ef3bb9
commit: f14cda9ee60a6e325331c0f9547fbdd98cdc5c88 [8155/8157] DEBUG: Dispcc
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/andersson/kernel/commit/f14cda9ee60a6e325331c0f9547fbdd98cdc5c88
        git remote add andersson-kernel https://github.com/andersson/kernel
        git fetch --no-tags andersson-kernel wip/for-robert/dsi-panel-fixup
        git checkout f14cda9ee60a6e325331c0f9547fbdd98cdc5c88
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

   In file included from include/linux/kernel.h:19,
                    from drivers/clk/qcom/clk-rcg2.c:6:
   drivers/clk/qcom/clk-rcg2.c: In function 'update_config':
>> drivers/clk/qcom/clk-rcg2.c:109:16: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'long unsigned int' [-Wformat=]
     109 |         printk("%s() regmap_update_bits(0x%08x, 0x%04x)", __func__, rcg->cmd_rcgr, CMD_UPDATE);
         |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:418:25: note: in definition of macro 'printk_index_wrap'
     418 |                 _p_func(_fmt, ##__VA_ARGS__);                           \
         |                         ^~~~
   drivers/clk/qcom/clk-rcg2.c:109:9: note: in expansion of macro 'printk'
     109 |         printk("%s() regmap_update_bits(0x%08x, 0x%04x)", __func__, rcg->cmd_rcgr, CMD_UPDATE);
         |         ^~~~~~
   drivers/clk/qcom/clk-rcg2.c:109:54: note: format string is defined here
     109 |         printk("%s() regmap_update_bits(0x%08x, 0x%04x)", __func__, rcg->cmd_rcgr, CMD_UPDATE);
         |                                                   ~~~^
         |                                                      |
         |                                                      unsigned int
         |                                                   %04lx
   In file included from include/linux/kernel.h:19,
                    from drivers/clk/qcom/clk-rcg2.c:6:
   drivers/clk/qcom/clk-rcg2.c:120:24: warning: format '%x' expects argument of type 'unsigned int', but argument 4 has type 'long unsigned int' [-Wformat=]
     120 |                 printk("%s() regmap_read(0x%08x, 0x%08x) = 0x%04x", __func__, rcg->cmd_rcgr, CMD_UPDATE, cmd);
         |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/printk.h:418:25: note: in definition of macro 'printk_index_wrap'
     418 |                 _p_func(_fmt, ##__VA_ARGS__);                           \
         |                         ^~~~
   drivers/clk/qcom/clk-rcg2.c:120:17: note: in expansion of macro 'printk'
     120 |                 printk("%s() regmap_read(0x%08x, 0x%08x) = 0x%04x", __func__, rcg->cmd_rcgr, CMD_UPDATE, cmd);
         |                 ^~~~~~
   drivers/clk/qcom/clk-rcg2.c:120:55: note: format string is defined here
     120 |                 printk("%s() regmap_read(0x%08x, 0x%08x) = 0x%04x", __func__, rcg->cmd_rcgr, CMD_UPDATE, cmd);
         |                                                    ~~~^
         |                                                       |
         |                                                       unsigned int
         |                                                    %08lx


vim +109 drivers/clk/qcom/clk-rcg2.c

    99	
   100	static int update_config(struct clk_rcg2 *rcg)
   101	{
   102		int count, ret;
   103		u32 cmd;
   104		struct clk_hw *hw = &rcg->clkr.hw;
   105		const char *name = clk_hw_get_name(hw);
   106	
   107		printk("%s() name=%s", __func__, name);
   108	
 > 109		printk("%s() regmap_update_bits(0x%08x, 0x%04x)", __func__, rcg->cmd_rcgr, CMD_UPDATE);
   110		ret = regmap_update_bits(rcg->clkr.regmap, rcg->cmd_rcgr + CMD_REG,
   111					 CMD_UPDATE, CMD_UPDATE);
   112		if (ret) {
   113			printk("%s() regmap_update_bits() failed", __func__);
   114			return ret;
   115		}
   116	
   117		/* Wait for update to take effect */
   118		for (count = 500; count > 0; count--) {
   119			ret = regmap_read(rcg->clkr.regmap, rcg->cmd_rcgr + CMD_REG, &cmd);
   120			printk("%s() regmap_read(0x%08x, 0x%08x) = 0x%04x", __func__, rcg->cmd_rcgr, CMD_UPDATE, cmd);
   121			if (ret) {
   122				printk("%s() regmap_read() failed", __func__);
   123				return ret;
   124			}
   125			if (!(cmd & CMD_UPDATE)) {
   126				printk("%s() !(cmd & CMD_UPDATE)", __func__);
   127				return 0;
   128			}
   129			udelay(1);
   130		}
   131	
   132		WARN(1, "%s: rcg didn't update its configuration.", name);
   133		return -EBUSY;
   134	}
   135	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip