[kbuild-all] [ti:ti-linux-5.4.y 7094/11671] drivers/crypto/sa2ul.c:1296:25: warning: variable 'rctx' set but not used

Hi Tero, FYI, the error/warning still remains. tree: git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git ti-linux-5.4.y head: a78602ef875fdf4abcb333da76f020cba57cc913 commit: 2e7c47e07621e99da76060d7e85b2ea050ad97b0 [7094/11671] crypto: sa2ul: convert hash fallback mechanism to use local buffer config: nios2-randconfig-r034-20201117 (attached as .config) compiler: nios2-linux-gcc (GCC) 9.3.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 git remote add ti git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git git fetch --no-tags ti ti-linux-5.4.y git checkout 2e7c47e07621e99da76060d7e85b2ea050ad97b0 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=nios2 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): drivers/crypto/sa2ul.c: In function 'sa_cipher_setkey': drivers/crypto/sa2ul.c:856:14: warning: variable 'cra_name' set but not used [-Wunused-but-set-variable] 856 | const char *cra_name; | ^~~~~~~~ drivers/crypto/sa2ul.c: In function 'sa_sha_dma_in_callback': 1296 | struct sa_sha_req_ctx *rctx; | ^~~~ In file included from include/linux/printk.h:327, from include/linux/kernel.h:15, from include/linux/clk.h:13, from drivers/crypto/sa2ul.c:11: drivers/crypto/sa2ul.c: In function 'sa_sha_init': drivers/crypto/sa2ul.c:1525:33: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] 1525 | crypto_ahash_digestsize(tfm), (u64)rctx); | ^ include/linux/dynamic_debug.h:125:15: note: in definition of macro '__dynamic_func_call' 125 | func(&id, ##__VA_ARGS__); \ | ^~~~~~~~~~~ include/linux/dynamic_debug.h:157:2: note: in expansion of macro '_dynamic_func_call' 157 | _dynamic_func_call(fmt,__dynamic_dev_dbg, \ | ^~~~~~~~~~~~~~~~~~ include/linux/device.h:1751:2: note: in expansion of macro 'dynamic_dev_dbg' 1751 | dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~~~~~~~~~ drivers/crypto/sa2ul.c:1524:2: note: in expansion of macro 'dev_dbg' 1524 | dev_dbg(sa_k3_dev, "init: digest size: %d, rctx=%llx\n", | ^~~~~~~ drivers/crypto/sa2ul.c: At top level: drivers/crypto/sa2ul.c:2211:6: warning: no previous prototype for 'sa_register_algos' [-Wmissing-prototypes] 2211 | void sa_register_algos(const struct device *dev) | ^~~~~~~~~~~~~~~~~ drivers/crypto/sa2ul.c:2243:6: warning: no previous prototype for 'sa_unregister_algos' [-Wmissing-prototypes] 2243 | void sa_unregister_algos(const struct device *dev) | ^~~~~~~~~~~~~~~~~~~ vim +/rctx +1296 drivers/crypto/sa2ul.c 491c6a44ee1ed98 Keerthy 2020-01-17 1289 b30be50bf38eb79 Tero Kristo 2020-05-26 1290 static void sa_sha_dma_in_callback(void *data) dfcac7a6a1f9e04 Keerthy 2020-01-17 1291 { dfcac7a6a1f9e04 Keerthy 2020-01-17 1292 struct sa_rx_data *rxd = (struct sa_rx_data *)data; b30be50bf38eb79 Tero Kristo 2020-05-26 1293 struct ahash_request *req; b30be50bf38eb79 Tero Kristo 2020-05-26 1294 struct crypto_ahash *tfm; b30be50bf38eb79 Tero Kristo 2020-05-26 1295 unsigned int authsize; b30be50bf38eb79 Tero Kristo 2020-05-26 @1296 struct sa_sha_req_ctx *rctx; dfcac7a6a1f9e04 Keerthy 2020-01-17 1297 int i, sg_nents; dfcac7a6a1f9e04 Keerthy 2020-01-17 1298 size_t ml, pl; dfcac7a6a1f9e04 Keerthy 2020-01-17 1299 u32 *mdptr, *result; dfcac7a6a1f9e04 Keerthy 2020-01-17 1300 b30be50bf38eb79 Tero Kristo 2020-05-26 1301 req = container_of(rxd->req, struct ahash_request, base); b30be50bf38eb79 Tero Kristo 2020-05-26 1302 tfm = crypto_ahash_reqtfm(req); b30be50bf38eb79 Tero Kristo 2020-05-26 1303 authsize = crypto_ahash_digestsize(tfm); b30be50bf38eb79 Tero Kristo 2020-05-26 1304 rctx = ahash_request_ctx(req); b30be50bf38eb79 Tero Kristo 2020-05-26 1305 dfcac7a6a1f9e04 Keerthy 2020-01-17 1306 mdptr = (u32 *)dmaengine_desc_get_metadata_ptr(rxd->tx_in, &pl, &ml); dfcac7a6a1f9e04 Keerthy 2020-01-17 1307 result = (u32 *)req->result; dfcac7a6a1f9e04 Keerthy 2020-01-17 1308 2e7c47e07621e99 Tero Kristo 2020-05-26 1309 if (result) dfcac7a6a1f9e04 Keerthy 2020-01-17 1310 for (i = 0; i < (authsize / 4); i++) dfcac7a6a1f9e04 Keerthy 2020-01-17 1311 result[i] = htonl(mdptr[i + 4]); dfcac7a6a1f9e04 Keerthy 2020-01-17 1312 b30be50bf38eb79 Tero Kristo 2020-05-26 1313 sg_nents = sg_nents_for_len(req->src, req->nbytes); b30be50bf38eb79 Tero Kristo 2020-05-26 1314 dma_unmap_sg(rxd->ddev, req->src, sg_nents, DMA_FROM_DEVICE); b30be50bf38eb79 Tero Kristo 2020-05-26 1315 b30be50bf38eb79 Tero Kristo 2020-05-26 1316 kfree(rxd->split_src_sg); dfcac7a6a1f9e04 Keerthy 2020-01-17 1317 dfcac7a6a1f9e04 Keerthy 2020-01-17 1318 kfree(rxd); dfcac7a6a1f9e04 Keerthy 2020-01-17 1319 dfcac7a6a1f9e04 Keerthy 2020-01-17 1320 ahash_request_complete(req, 0); dfcac7a6a1f9e04 Keerthy 2020-01-17 1321 } dfcac7a6a1f9e04 Keerthy 2020-01-17 1322 :::::: The code at line 1296 was first introduced by commit :::::: b30be50bf38eb7964bc59dbfefd120939b357b37 crypto: sa2ul: sync codebase with latest upstream :::::: TO: Tero Kristo <t-kristo(a)ti.com&gt; :::::: CC: Tero Kristo <t-kristo(a)ti.com&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org