From: Sean Anderson Date: Tue, 22 Nov 2022 17:54:51 +0000 (-0500) Subject: x86: fsp: Only compile fsp_save_s3_stack if (SPL_)DM_RTC is enabled X-Git-Tag: v2025.01-rc5-pxa1908~1158^2~10^2~13 X-Git-Url: http://git.dujemihanovic.xyz/contact?a=commitdiff_plain;h=0db588caf99821b616169a7888d8a9515604940c;p=u-boot.git x86: fsp: Only compile fsp_save_s3_stack if (SPL_)DM_RTC is enabled This function calls rtc_write32, which has a different signature depending on if (SPL_)DM_RTC is enabled or not. This could result in a mismatch in SPL if DM_RTC was enabled but SPL_DM_RTC, as the non-DM declaration would still be used in SPL even though the implementation would be for non-DM_RTC. We are switching to the correct definitions in the next commit, so this will become a compilation error. Since fsp_save_s3_stack is not called from SPL, avoid compiling it if (SPL_)DM_RTC is disabled. Signed-off-by: Sean Anderson --- diff --git a/arch/x86/lib/fsp/fsp_common.c b/arch/x86/lib/fsp/fsp_common.c index 82f7d3ab5f..8f2977a807 100644 --- a/arch/x86/lib/fsp/fsp_common.c +++ b/arch/x86/lib/fsp/fsp_common.c @@ -61,6 +61,7 @@ void board_final_init(void) debug("OK\n"); } +#if CONFIG_IS_ENABLED(DM_RTC) int fsp_save_s3_stack(void) { struct udevice *dev; @@ -84,3 +85,4 @@ int fsp_save_s3_stack(void) return 0; } +#endif