Hi Kiseok,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on broonie-sound/for-next] [also build test WARNING on robh/for-next linus/master v6.2-rc2 next-20221226] [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/Kiseok-Jo/ASoC-sma1303-Add-dr... base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next patch link: https://lore.kernel.org/r/20230104064342.2094-4-kiseok.jo%40irondevice.com patch subject: [PATCH 3/3] ASOC: sma1303: change the overall contents config: s390-allyesconfig compiler: s390-linux-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/287cf116194f9c120409ae8a64f719... git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Kiseok-Jo/ASoC-sma1303-Add-driver-for-Iron-Device-SMA1303-Amp/20230104-150052 git checkout 287cf116194f9c120409ae8a64f719fd7696e2cc # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=s390 olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=s390 SHELL=/bin/bash sound/soc/codecs/
If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot lkp@intel.com
All warnings (new ones prefixed by >>):
In file included from include/linux/device.h:15, from include/linux/acpi.h:15, from include/linux/i2c.h:13, from sound/soc/codecs/sma1303.c:12: sound/soc/codecs/sma1303.c: In function 'sma1303_add_component_controls':
sound/soc/codecs/sma1303.c:708:32: warning: format '%d' expects argument of type 'int', but argument 5 has type 'long unsigned int' [-Wformat=]
708 | dev_info(sma1303->dev, "%s : prefix="%s", num=%d, ret=%d\n", __func__, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt' 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~ sound/soc/codecs/sma1303.c:708:9: note: in expansion of macro 'dev_info' 708 | dev_info(sma1303->dev, "%s : prefix="%s", num=%d, ret=%d\n", __func__, | ^~~~~~~~ sound/soc/codecs/sma1303.c:708:58: note: format string is defined here 708 | dev_info(sma1303->dev, "%s : prefix="%s", num=%d, ret=%d\n", __func__, | ~^ | | | int | %ld
vim +708 sound/soc/codecs/sma1303.c
675 676 static int sma1303_add_component_controls(struct snd_soc_component *component) 677 { 678 struct sma1303_priv *sma1303 = snd_soc_component_get_drvdata(component); 679 unsigned char **name; 680 int index, ret=0; 681 682 sma1303_controls = devm_kzalloc(sma1303->dev, 683 sizeof(sma1303_snd_controls), GFP_KERNEL); 684 name = devm_kzalloc(sma1303->dev, 685 ARRAY_SIZE(sma1303_snd_controls), GFP_KERNEL); 686 687 for (index=0; index<ARRAY_SIZE(sma1303_snd_controls); index++) { 688 sma1303_controls[index] = sma1303_snd_controls[index]; 689 name[index] = devm_kzalloc(sma1303->dev, 690 MAX_CONTROL_NAME, GFP_KERNEL); 691 if (!name[index]) { 692 dev_err(sma1303->dev, "%s : No Memory - %s\n", 693 __func__, sma1303_snd_controls[index].name); 694 sma1303_controls[index].name = 695 sma1303_snd_controls[index].name; 696 } 697 else { 698 scnprintf(name[index], MAX_CONTROL_NAME, "%s %s", 699 sma1303->dev->driver->name, 700 sma1303_snd_controls[index].name); 701 sma1303_controls[index].name = name[index]; 702 } 703 } 704 705 ret = snd_soc_add_component_controls(component, 706 sma1303_controls, ARRAY_SIZE(sma1303_snd_controls)); 707
708 dev_info(sma1303->dev, "%s : prefix="%s", num=%d, ret=%d\n", __func__,
709 sma1303->dev->driver->name, 710 ARRAY_SIZE(sma1303_snd_controls), ret); 711 712 return ret; 713