[alsa-devel] [asoc:for-5.4 175/179] sound/soc/intel/haswell/sst-haswell-ipc.c:2018:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}'

kbuild test robot lkp at intel.com
Thu Aug 8 08:09:27 CEST 2019


tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/broonie/sound.git for-5.4
head:   a860fac420971c5a90d4f78959b44ead793aee4f
commit: abf31feea26c0f412a191c83f408311a0de7435c [175/179] ASoC: Intel: Update request-reply IPC model
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.4.0-10) 7.4.0
reproduce:
        git checkout abf31feea26c0f412a191c83f408311a0de7435c
        # save the attached .config to linux build tree
        make ARCH=i386 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp at intel.com>

All warnings (new ones prefixed by >>):

   In file included from include/linux/printk.h:332:0,
                    from include/linux/kernel.h:15,
                    from sound/soc/intel/haswell/sst-haswell-ipc.c:9:
   sound/soc/intel/haswell/sst-haswell-ipc.c: In function 'sst_hsw_module_set_param':
>> sound/soc/intel/haswell/sst-haswell-ipc.c:2018:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
      dev_dbg(dev, "transfer parameter size : %lu\n",
                   ^
   include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call'
      func(&id, ##__VA_ARGS__);  \
                  ^~~~~~~~~~~
   include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call'
     _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
     ^~~~~~~~~~~~~~~~~~
   include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg'
     dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
     ^~~~~~~~~~~~~~~
   include/linux/device.h:1503:23: note: in expansion of macro 'dev_fmt'
     dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
                          ^~~~~~~
>> sound/soc/intel/haswell/sst-haswell-ipc.c:2018:3: note: in expansion of macro 'dev_dbg'
      dev_dbg(dev, "transfer parameter size : %lu\n",
      ^~~~~~~
   sound/soc/intel/haswell/sst-haswell-ipc.c:2022:16: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
      dev_dbg(dev, "transfer parameter aligned size : %lu\n",
                   ^
   include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call'
      func(&id, ##__VA_ARGS__);  \
                  ^~~~~~~~~~~
   include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call'
     _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
     ^~~~~~~~~~~~~~~~~~
   include/linux/device.h:1503:2: note: in expansion of macro 'dynamic_dev_dbg'
     dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
     ^~~~~~~~~~~~~~~
   include/linux/device.h:1503:23: note: in expansion of macro 'dev_fmt'
     dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
                          ^~~~~~~
   sound/soc/intel/haswell/sst-haswell-ipc.c:2022:3: note: in expansion of macro 'dev_dbg'
      dev_dbg(dev, "transfer parameter aligned size : %lu\n",
      ^~~~~~~

vim +2018 sound/soc/intel/haswell/sst-haswell-ipc.c

  1993	
  1994	int sst_hsw_module_set_param(struct sst_hsw *hsw,
  1995		u32 module_id, u32 instance_id, u32 parameter_id,
  1996		u32 param_size, char *param)
  1997	{
  1998		int ret;
  1999		struct sst_ipc_message request = {0};
  2000		u32 payload_size = 0;
  2001		struct sst_hsw_transfer_parameter *parameter;
  2002		struct device *dev = hsw->dev;
  2003	
  2004		request.header = IPC_GLB_TYPE(IPC_GLB_MODULE_OPERATION) |
  2005				IPC_MODULE_OPERATION(IPC_MODULE_SET_PARAMETER) |
  2006				IPC_MODULE_ID(module_id);
  2007		dev_dbg(dev, "sst_hsw_module_set_param header=%x\n",
  2008				(u32)request.header);
  2009	
  2010		payload_size = param_size +
  2011			sizeof(struct sst_hsw_transfer_parameter) -
  2012			sizeof(struct sst_hsw_transfer_list);
  2013		dev_dbg(dev, "parameter size : %d\n", param_size);
  2014		dev_dbg(dev, "payload size   : %d\n", payload_size);
  2015	
  2016		if (payload_size <= SST_HSW_IPC_MAX_SHORT_PARAMETER_SIZE) {
  2017			/* short parameter, mailbox can contain data */
> 2018			dev_dbg(dev, "transfer parameter size : %lu\n",
  2019				request.size);
  2020	
  2021			request.size = ALIGN(payload_size, 4);
  2022			dev_dbg(dev, "transfer parameter aligned size : %lu\n",
  2023				request.size);
  2024	
  2025			parameter = kzalloc(request.size, GFP_KERNEL);
  2026			if (parameter == NULL)
  2027				return -ENOMEM;
  2028	
  2029			memcpy(parameter->data, param, param_size);
  2030		} else {
  2031			dev_warn(dev, "transfer parameter size too large!");
  2032			return 0;
  2033		}
  2034	
  2035		parameter->parameter_id = parameter_id;
  2036		parameter->data_size = param_size;
  2037		request.data = parameter;
  2038	
  2039		ret = sst_ipc_tx_message_wait(&hsw->ipc, request, NULL);
  2040		if (ret < 0)
  2041			dev_err(dev, "ipc: module set parameter failed - %d\n", ret);
  2042	
  2043		kfree(parameter);
  2044	
  2045		return ret;
  2046	}
  2047	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 69464 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20190808/6a8824e5/attachment-0001.gz>


More information about the Alsa-devel mailing list