[thesofproject:topic/sof-dev-rebase 36/83] drivers/soundwire/debugfs.c:300:30: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'size_t' {aka 'unsigned int'}

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

tree:   https://github.com/thesofproject/linux topic/sof-dev-rebase
head:   4dbf0a74815675188836424b784024cdf13365f9
commit: feae4ee142eeeccad34203a42a4d63019a46062c [36/83] soundwire: debugfs: add interface for BPT/BRA transfers
config: arm-randconfig-003-20250123 (https://download.01.org/0day-ci/archive/20250123/202501231507.gOqcfPTX-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250123/202501231507.gOqcfPTX-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/202501231507.gOqcfPTX-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/printk.h:610,
                    from include/asm-generic/bug.h:22,
                    from arch/arm/include/asm/bug.h:60,
                    from include/linux/bug.h:5,
                    from include/linux/thread_info.h:13,
                    from include/linux/sched.h:14,
                    from include/linux/ratelimit.h:6,
                    from include/linux/dev_printk.h:16,
                    from include/linux/device.h:15,
                    from drivers/soundwire/debugfs.c:5:
   drivers/soundwire/debugfs.c: In function 'cmd_go':
>> drivers/soundwire/debugfs.c:300:30: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
     300 |         dev_dbg(&slave->dev, "command completed, num_byte %ld status %d, time %lld ms\n",
         |                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:224:29: note: in definition of macro '__dynamic_func_call_cls'
     224 |                 func(&id, ##__VA_ARGS__);                       \
         |                             ^~~~~~~~~~~
   include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
     250 |         _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
         |         ^~~~~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:273:9: note: in expansion of macro '_dynamic_func_call'
     273 |         _dynamic_func_call(fmt, __dynamic_dev_dbg,              \
         |         ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:165:9: note: in expansion of macro 'dynamic_dev_dbg'
     165 |         dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |         ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:165:30: note: in expansion of macro 'dev_fmt'
     165 |         dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                              ^~~~~~~
   drivers/soundwire/debugfs.c:300:9: note: in expansion of macro 'dev_dbg'
     300 |         dev_dbg(&slave->dev, "command completed, num_byte %ld status %d, time %lld ms\n",
         |         ^~~~~~~
   drivers/soundwire/debugfs.c:300:61: note: format string is defined here
     300 |         dev_dbg(&slave->dev, "command completed, num_byte %ld status %d, time %lld ms\n",
         |                                                           ~~^
         |                                                             |
         |                                                             long int
         |                                                           %d


vim +300 drivers/soundwire/debugfs.c

   234	
   235	static int cmd_go(void *data, u64 value)
   236	{
   237		const struct firmware *fw = NULL;
   238		struct sdw_slave *slave = data;
   239		ktime_t start_t;
   240		ktime_t finish_t;
   241		int ret;
   242	
   243		if (value != 1)
   244			return -EINVAL;
   245	
   246		/* one last check */
   247		if (start_addr > SDW_REG_MAX ||
   248		    num_bytes == 0 || num_bytes > MAX_CMD_BYTES)
   249			return -EINVAL;
   250	
   251		ret = pm_runtime_get_sync(&slave->dev);
   252		if (ret < 0 && ret != -EACCES) {
   253			pm_runtime_put_noidle(&slave->dev);
   254			return ret;
   255		}
   256	
   257		if (cmd == 0) {
   258			ret = request_firmware(&fw, firmware_file, &slave->dev);
   259			if (ret < 0) {
   260				dev_err(&slave->dev, "firmware %s not found\n", firmware_file);
   261				goto out;
   262			}
   263			if (fw->size < num_bytes) {
   264				dev_err(&slave->dev,
   265					"firmware %s: firmware size %zd, desired %zd\n",
   266					firmware_file, fw->size, num_bytes);
   267				goto out;
   268			}
   269		}
   270	
   271		/* Userspace changed the hardware state behind the kernel's back */
   272		add_taint(TAINT_USER, LOCKDEP_STILL_OK);
   273	
   274		dev_dbg(&slave->dev, "starting command\n");
   275		start_t = ktime_get();
   276	
   277		if (cmd == 0) {
   278			if (cmd_type)
   279				ret = do_bpt_sequence(slave, true, (u8 *)fw->data);
   280			else
   281				ret = sdw_nwrite_no_pm(slave, start_addr, num_bytes, fw->data);
   282		} else {
   283			memset(read_buffer, 0, sizeof(read_buffer));
   284	
   285			if (cmd_type)
   286				ret = do_bpt_sequence(slave, false, read_buffer);
   287			else
   288				ret = sdw_nread_no_pm(slave, start_addr, num_bytes, read_buffer);
   289		}
   290	
   291		finish_t = ktime_get();
   292	
   293	out:
   294		if (fw)
   295			release_firmware(fw);
   296	
   297		pm_runtime_mark_last_busy(&slave->dev);
   298		pm_runtime_put(&slave->dev);
   299	
 > 300		dev_dbg(&slave->dev, "command completed, num_byte %ld status %d, time %lld ms\n",
   301			num_bytes, ret, div_u64(finish_t - start_t, NSEC_PER_MSEC));
   302	
   303		return ret;
   304	}
   305	DEFINE_DEBUGFS_ATTRIBUTE(cmd_go_fops, NULL,
   306				 cmd_go, "%llu\n");
   307	

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