Re: [PATCH v12 4/5] firmware: imx: add driver for NXP EdgeLock Enclave

kernel test robot
2025-01-20T19:43:57Z

Hi Pankaj,

kernel test robot noticed the following build warnings:

[auto build test WARNING on ffd294d346d185b70e28b1a28abe367bbfe53c04]

url: https://github.com/intel-lab-lkp/linux/commits/Pankaj-Gupta/Documentation-firmware-add-imx-se-to-other_interfaces/20250120-193050
base: ffd294d346d185b70e28b1a28abe367bbfe53c04
patch link: https://lore.kernel.org/r/20250120-imx-se-if-v12-4-c5ec9754570c%40nxp.com
patch subject: [PATCH v12 4/5] firmware: imx: add driver for NXP EdgeLock Enclave
config: arm64-randconfig-002-20250121 (https://download.01.org/0day-ci/archive/20250121/202501210346.wYxcOI66-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250121/202501210346.wYxcOI66-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202501210346.wYxcOI66-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/firmware/imx/se_ctrl.c:505:12: warning: 'se_resume' defined but not used [-Wunused-function]
505 | static int se_resume(struct device *dev)
| ^~~~~~~~~
>> drivers/firmware/imx/se_ctrl.c:491:12: warning: 'se_suspend' defined but not used [-Wunused-function]
491 | static int se_suspend(struct device *dev)
| ^~~~~~~~~~
drivers/firmware/imx/se_ctrl.c:228:12: warning: 'se_load_firmware' defined but not used [-Wunused-function]
228 | static int se_load_firmware(struct se_if_priv *priv)
| ^~~~~~~~~~~~~~~~

vim +/se_resume +505 drivers/firmware/imx/se_ctrl.c

490
> 491 static int se_suspend(struct device *dev)
492 {
493 struct se_if_priv *priv = dev_get_drvdata(dev);
494 struct se_fw_load_info *load_fw;
495 int ret = 0;
496
497 load_fw = get_load_fw_instance(priv);
498
499 if (load_fw->imem_mgmt)
500 ret = se_save_imem_state(priv, &load_fw->imem);
501
502 return ret;
503 }
504
> 505 static int se_resume(struct device *dev)
506 {
507 struct se_if_priv *priv = dev_get_drvdata(dev);
508 struct se_fw_load_info *load_fw;
509
510 load_fw = get_load_fw_instance(priv);
511
512 if (load_fw->imem_mgmt)
513 se_restore_imem_state(priv, &load_fw->imem);
514
515 return 0;
516 }
517

--
0-DAY CI Kernel Test Service