Hi SEO, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on jejb-scsi/for-next] [also build test WARNING on mkp-scsi/for-next krzk/for-next linus/master v6.0-rc5 next-20220912] [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/SEO-HOYOUNG/scsi-ufs-add-a-variant-operation-in-struct-ufs_hba_variant_ops/20220913-101855 base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next config: loongarch-allyesconfig (https://download.01.org/0day-ci/archive/20220913/202209131145.uQlsf9Uv-lkp@xxxxxxxxx/config) compiler: loongarch64-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/60f9cc2a287e2bfe58c8355519797a9071b00afa git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review SEO-HOYOUNG/scsi-ufs-add-a-variant-operation-in-struct-ufs_hba_variant_ops/20220913-101855 git checkout 60f9cc2a287e2bfe58c8355519797a9071b00afa # 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=loongarch SHELL=/bin/bash drivers/ufs/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): drivers/ufs/host/ufs-exynos.c: In function 'exynos_ufs_check_int_error': drivers/ufs/host/ufs-exynos.c:1388:17: error: 'val' undeclared (first use in this function) 1388 | val = hci_readl(ufs, HCI_AH8_STATE); | ^~~ drivers/ufs/host/ufs-exynos.c:1388:17: note: each undeclared identifier is reported only once for each function it appears in >> drivers/ufs/host/ufs-exynos.c:1392:40: warning: assignment to 'bool *' {aka '_Bool *'} from 'int' makes pointer from integer without a cast [-Wint-conversion] 1392 | queue_eh_work = true; | ^ >> drivers/ufs/host/ufs-exynos.c:1383:67: warning: parameter 'queue_eh_work' set but not used [-Wunused-but-set-parameter] 1383 | static void exynos_ufs_check_int_error(struct ufs_hba *hba, bool *queue_eh_work) | ~~~~~~^~~~~~~~~~~~~ vim +1392 drivers/ufs/host/ufs-exynos.c 1382 > 1383 static void exynos_ufs_check_int_error(struct ufs_hba *hba, bool *queue_eh_work) 1384 { 1385 struct exynos_ufs *ufs = ufshcd_get_variant(hba); 1386 1387 if (ufshcd_is_auto_hibern8_supported(hba)) { 1388 val = hci_readl(ufs, HCI_AH8_STATE); 1389 1390 if (val & HCI_AH8_STATE_ERROR) { 1391 ufshcd_set_link_broken(hba); > 1392 queue_eh_work = true; 1393 } 1394 } 1395 } 1396 -- 0-DAY CI Kernel Test Service https://01.org/lkp
