Hi Corentin, I love your patch! Perhaps something to improve: [auto build test WARNING on rockchip/for-next] [also build test WARNING on herbert-cryptodev-2.6/master herbert-crypto-2.6/master v5.17-rc6 next-20220228] [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] url: https://github.com/0day-ci/linux/commits/Corentin-Labbe/crypto-rockchip-permit-to-pass-self-tests/20220301-035430 base: https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git for-next config: arm64-randconfig-r006-20220228 (https://download.01.org/0day-ci/archive/20220301/202203011017.qqb0tTar-lkp@xxxxxxxxx/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e) 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 arm64 cross compiling tool for clang build # apt-get install binutils-aarch64-linux-gnu # https://github.com/0day-ci/linux/commit/03fb353165f76dc7c5ed7c27401537552070406b git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Corentin-Labbe/crypto-rockchip-permit-to-pass-self-tests/20220301-035430 git checkout 03fb353165f76dc7c5ed7c27401537552070406b # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/crypto/rockchip/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): >> drivers/crypto/rockchip/rk3288_crypto.c:101:6: warning: data argument not used by format string [-Wformat-extra-args] rk_cipher_algs[i]->alg.skcipher.base.cra_driver_name); ^ 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__); \ ~~~ ^ 1 warning generated. vim +101 drivers/crypto/rockchip/rk3288_crypto.c 89 90 static int rk_crypto_register(struct rk_crypto_info *crypto_info) 91 { 92 unsigned int i, k; 93 int err = 0; 94 95 for (i = 0; i < ARRAY_SIZE(rk_cipher_algs); i++) { 96 rk_cipher_algs[i]->dev = crypto_info; 97 switch (rk_cipher_algs[i]->type) { 98 case CRYPTO_ALG_TYPE_SKCIPHER: 99 dev_info(crypto_info->dev, "Register %s as\n", 100 rk_cipher_algs[i]->alg.skcipher.base.cra_name, > 101 rk_cipher_algs[i]->alg.skcipher.base.cra_driver_name); 102 err = crypto_register_skcipher(&rk_cipher_algs[i]->alg.skcipher); 103 break; 104 case CRYPTO_ALG_TYPE_AHASH: 105 dev_info(crypto_info->dev, "Register %s as %s\n", 106 rk_cipher_algs[i]->alg.hash.halg.base.cra_name, 107 rk_cipher_algs[i]->alg.hash.halg.base.cra_driver_name); 108 err = crypto_register_ahash(&rk_cipher_algs[i]->alg.hash); 109 break; 110 default: 111 dev_err(crypto_info->dev, "unknown algorithm\n"); 112 } 113 if (err) 114 goto err_cipher_algs; 115 } 116 return 0; 117 118 err_cipher_algs: 119 for (k = 0; k < i; k++) { 120 if (rk_cipher_algs[i]->type == CRYPTO_ALG_TYPE_SKCIPHER) 121 crypto_unregister_skcipher(&rk_cipher_algs[k]->alg.skcipher); 122 else 123 crypto_unregister_ahash(&rk_cipher_algs[i]->alg.hash); 124 } 125 return err; 126 } 127 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
