Building with CONFIG_FW_LOADER=m && CONFIG_FW_UPLOAD=y causes errors: ld: drivers/fpga/intel-m10-bmc-sec-update.o: in function `m10bmc_sec_remove': intel-m10-bmc-sec-update.c:(.text+0x25b): undefined reference to `firmware_upload_unregister' ld: drivers/fpga/intel-m10-bmc-sec-update.o: in function `m10bmc_sec_probe': intel-m10-bmc-sec-update.c:(.text+0x3c1): undefined reference to `firmware_upload_register' When CONFIG_FW_LOADER is configed with module, the firmware_upload_unregister() could not be seen in m10bmc_sec_remove(), it is controlled by CONFIG_FW_UPLOAD that it depends CONFIG_FW_LOADER. We fix it by adding FW_LOADER in FPGA_M10_BMC_SEC_UPDATE depends. Fixes: 5cd339b370e2 ("fpga: m10bmc-sec: add max10 secure update functions") Signed-off-by: Zhang Qilong <zhangqilong3@xxxxxxxxxx> --- drivers/fpga/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig index 6c416955da53..cfc684e9be9a 100644 --- a/drivers/fpga/Kconfig +++ b/drivers/fpga/Kconfig @@ -246,7 +246,7 @@ config FPGA_MGR_VERSAL_FPGA config FPGA_M10_BMC_SEC_UPDATE tristate "Intel MAX10 BMC Secure Update driver" - depends on MFD_INTEL_M10_BMC && FW_UPLOAD + depends on MFD_INTEL_M10_BMC && FW_UPLOAD && FW_LOADER help Secure update support for the Intel MAX10 board management controller. -- 2.25.1