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: riscv-randconfig-r042-20230102 compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 7a8cb6cd4e3ff8aaadebff2b9d3ee9e2a326d444) 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 # install riscv cross compiling tool for clang build # apt-get install binutils-riscv64-linux-gnu # 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=clang make.cross W=1 O=build_dir ARCH=riscv olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv 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 >>):
sound/soc/codecs/sma1303.c:710:4: warning: format specifies type 'int' but the argument has type 'unsigned long' [-Wformat]
ARRAY_SIZE(sma1303_snd_controls), ret); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/dev_printk.h:150:67: note: expanded from macro 'dev_info' dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__) ~~~ ^~~~~~~~~~~ include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap' _p_func(dev, fmt, ##__VA_ARGS__); \ ~~~ ^~~~~~~~~~~ include/linux/kernel.h:55:25: note: expanded from macro 'ARRAY_SIZE' #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr)) ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated.
vim +710 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