Re: [PATCH 5/8] firmware: arm_scmi: Make SMC transport a standalone driver

From: kernel test robot
Date: Tue Jul 09 2024 - 23:38:37 EST


Hi Cristian,

kernel test robot noticed the following build warnings:

[auto build test WARNING on soc/for-next]
[also build test WARNING on next-20240709]
[cannot apply to linus/master v6.10-rc7]
[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/Cristian-Marussi/firmware-arm_scmi-Introduce-setup_shmem_iomap/20240707-082513
base: https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git for-next
patch link: https://lore.kernel.org/r/20240707002055.1835121-6-cristian.marussi%40arm.com
patch subject: [PATCH 5/8] firmware: arm_scmi: Make SMC transport a standalone driver
config: arm64-randconfig-r131-20240709 (https://download.01.org/0day-ci/archive/20240710/202407101157.p2OvSFcO-lkp@xxxxxxxxx/config)
compiler: aarch64-linux-gcc (GCC) 13.2.0
reproduce: (https://download.01.org/0day-ci/archive/20240710/202407101157.p2OvSFcO-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202407101157.p2OvSFcO-lkp@xxxxxxxxx/

sparse warnings: (new ones prefixed by >>)
>> drivers/firmware/arm_scmi/scmi_transport_smc.c:276:24: sparse: sparse: symbol 'scmi_smc_desc' was not declared. Should it be static?

vim +/scmi_smc_desc +276 drivers/firmware/arm_scmi/scmi_transport_smc.c

1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 275
1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 @276 const struct scmi_desc scmi_smc_desc = {
1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 277 .ops = &scmi_smc_ops,
1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 278 .max_rx_timeout_ms = 30,
7adb2c8aaaa6a3 drivers/firmware/arm_scmi/smc.c Etienne Carriere 2020-10-08 279 .max_msg = 20,
1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 280 .max_msg_size = 128,
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 281 /*
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 282 * Setting .sync_cmds_atomic_replies to true for SMC assumes that,
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 283 * once the SMC instruction has completed successfully, the issued
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 284 * SCMI command would have been already fully processed by the SCMI
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 285 * platform firmware and so any possible response value expected
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 286 * for the issued command will be immmediately ready to be fetched
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 287 * from the shared memory area.
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 288 */
117542b81fe7b1 drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 289 .sync_cmds_completed_on_ret = true,
0bfdca8a8661aa drivers/firmware/arm_scmi/smc.c Cristian Marussi 2021-12-20 290 .atomic_enabled = IS_ENABLED(CONFIG_ARM_SCMI_TRANSPORT_SMC_ATOMIC_ENABLE),
1dc6558062dadf drivers/firmware/arm_scmi/smc.c Peng Fan 2020-03-08 291 };
728a057b6ab114 drivers/firmware/arm_scmi/scmi_transport_smc.c Cristian Marussi 2024-07-07 292

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki