Hi,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on broonie-sound/for-next] [also build test ERROR on linus/master v6.1-rc4 next-20221111] [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/wangweidong-a-awinic-com/ASoC... base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next patch link: https://lore.kernel.org/r/1668165992-6493-7-git-send-email-wangweidong.a%40a... patch subject: [PATCH V3 6/6] ASoC:codecs:aw883xx corresponds to the modified Makefile and Kconfig config: nios2-randconfig-r036-20221111 compiler: nios2-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/b8778077cd1bc1375124d18df7afee... git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review wangweidong-a-awinic-com/ASoC-codecs-Add-i2c-and-codec-registration-for/20221111-202637 git checkout b8778077cd1bc1375124d18df7afeec32dca1950 # 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=nios2 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 error/warnings (new ones prefixed by >>):
sound/soc/codecs/aw883xx/aw883xx.c: In function 'reg_store':
sound/soc/codecs/aw883xx/aw883xx.c:1095:9: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
1095 | if (sscanf(buf, "%x %x", &databuf[0], &databuf[1]) == 2) | ^~ sound/soc/codecs/aw883xx/aw883xx.c:1097:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 1097 | regmap_write(aw883xx->regmap, databuf[0], databuf[1]); | ^~~~~~~~~~~~ sound/soc/codecs/aw883xx/aw883xx.c: At top level:
sound/soc/codecs/aw883xx/aw883xx.c:1799:19: error: initialization of 'void (*)(struct i2c_client *)' from incompatible pointer type 'int (*)(struct i2c_client *)' [-Werror=incompatible-pointer-types]
1799 | .remove = aw883xx_i2c_remove, | ^~~~~~~~~~~~~~~~~~ sound/soc/codecs/aw883xx/aw883xx.c:1799:19: note: (near initialization for 'aw883xx_i2c_driver.remove') In file included from sound/soc/codecs/aw883xx/aw883xx.c:30: sound/soc/codecs/aw883xx/aw883xx_pid_2049_reg.h:110:28: warning: 'aw_pid_2049_reg_access' defined but not used [-Wunused-const-variable=] 110 | static const unsigned char aw_pid_2049_reg_access[AW_PID_2049_REG_MAX] = { | ^~~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors --
sound/soc/codecs/aw883xx/aw883xx_device.c:634:5: warning: no previous prototype for 'aw883xx_dev_get_int_status' [-Wmissing-prototypes]
634 | int aw883xx_dev_get_int_status(struct aw_device *aw_dev, uint16_t *int_status) | ^~~~~~~~~~~~~~~~~~~~~~~~~~
sound/soc/codecs/aw883xx/aw883xx_device.c:650:6: warning: no previous prototype for 'aw883xx_dev_clear_int_status' [-Wmissing-prototypes]
650 | void aw883xx_dev_clear_int_status(struct aw_device *aw_dev) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
sound/soc/codecs/aw883xx/aw883xx_device.c:1000:5: warning: no previous prototype for 'aw_dev_get_ra' [-Wmissing-prototypes]
1000 | int aw_dev_get_ra(struct aw_cali_desc *cali_desc) | ^~~~~~~~~~~~~
sound/soc/codecs/aw883xx/aw883xx_device.c:1062:5: warning: no previous prototype for 'aw_dev_dsp_fw_update' [-Wmissing-prototypes]
1062 | int aw_dev_dsp_fw_update(struct aw_device *aw_dev, | ^~~~~~~~~~~~~~~~~~~~
vim +1799 sound/soc/codecs/aw883xx/aw883xx.c
00cbb9c2078dff Weidong Wang 2022-11-11 1791 00cbb9c2078dff Weidong Wang 2022-11-11 1792 static struct i2c_driver aw883xx_i2c_driver = { 00cbb9c2078dff Weidong Wang 2022-11-11 1793 .driver = { 00cbb9c2078dff Weidong Wang 2022-11-11 1794 .name = AW883XX_I2C_NAME, 00cbb9c2078dff Weidong Wang 2022-11-11 1795 .owner = THIS_MODULE, 00cbb9c2078dff Weidong Wang 2022-11-11 1796 .of_match_table = of_match_ptr(aw883xx_dt_match), 00cbb9c2078dff Weidong Wang 2022-11-11 1797 }, 00cbb9c2078dff Weidong Wang 2022-11-11 1798 .probe = aw883xx_i2c_probe, 00cbb9c2078dff Weidong Wang 2022-11-11 @1799 .remove = aw883xx_i2c_remove, 00cbb9c2078dff Weidong Wang 2022-11-11 1800 .id_table = aw883xx_i2c_id, 00cbb9c2078dff Weidong Wang 2022-11-11 1801 }; 00cbb9c2078dff Weidong Wang 2022-11-11 1802 module_i2c_driver(aw883xx_i2c_driver); 00cbb9c2078dff Weidong Wang 2022-11-11 1803