Hi Trevor,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on broonie-sound/for-next] [also build test WARNING on broonie-spi/for-next tiwai-sound/for-next tiwai-sound/for-linus linus/master v6.1-rc8 next-20221208] [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/Trevor-Wu/ASoC-mediatek-Add-s... base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next patch link: https://lore.kernel.org/r/20221208033148.21866-12-trevor.wu%40mediatek.com patch subject: [PATCH v3 11/12] ASoC: mediatek: mt8188: add machine driver with mt6359 config: microblaze-randconfig-s042-20221211 compiler: microblaze-linux-gcc (GCC) 12.1.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # apt-get install sparse # sparse version: v0.6.4-39-gce1a6720-dirty # https://github.com/intel-lab-lkp/linux/commit/dab49faa06751591b72eafac0c4bf9... git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Trevor-Wu/ASoC-mediatek-Add-support-for-MT8188-SoC/20221208-113341 git checkout dab49faa06751591b72eafac0c4bf98de9882cfa # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=microblaze SHELL=/bin/bash sound/soc/mediatek/mt8188/
If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot lkp@intel.com
sparse warnings: (new ones prefixed by >>)
sound/soc/mediatek/mt8188/mt8188-mt6359.c:388:33: sparse: sparse: incorrect type in argument 3 (different base types) @@ expected unsigned int to @@ got restricted snd_pcm_format_t [usertype] @@
sound/soc/mediatek/mt8188/mt8188-mt6359.c:388:33: sparse: expected unsigned int to sound/soc/mediatek/mt8188/mt8188-mt6359.c:388:33: sparse: got restricted snd_pcm_format_t [usertype]
vim +388 sound/soc/mediatek/mt8188/mt8188-mt6359.c
382 383 static int mt8188_dptx_hw_params_fixup(struct snd_soc_pcm_runtime *rtd, 384 struct snd_pcm_hw_params *params) 385 { 386 /* fix BE i2s format to 32bit, clean param mask first */ 387 snd_mask_reset_range(hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMAT),
388 0, SNDRV_PCM_FORMAT_LAST);
389 390 params_set_format(params, SNDRV_PCM_FORMAT_S32_LE); 391 392 return 0; 393 } 394