[PATCH v3 1/2] x86: fsp: Depend on DM_RTC

[Bin Meng]

On Mon, Oct 24, 2022 at 11:42 PM Sean Anderson <sean.anderson at seco.com> wrote:
>
> FSP support requires DM_RTC for rtc_write32. Select it.
>
> Fixes: ba65808e7d0 ("x86: fsp: Save stack address to CMOS for next S3 boot")
> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
> ---
> This seems like it would never have worked. Does fsp_save_s3_stack even

This was working before. Did you test it on x86 that now it is broken?

+Stefan

> get called in SPL? Maybe it should be converted to use dm_rtc_write
> instead.
>
> Changes in v3:
> - New
>
>  arch/x86/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index 7cbfd6c9720..ed8216d9ad0 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -362,6 +362,8 @@ config HAVE_FSP
>         depends on !EFI
>         select USE_HOB
>         select HAS_ROM
> +       select DM_RTC
> +       select SPL_DM_RTC
>         help
>           Select this option to add an Firmware Support Package binary to
>           the resulting U-Boot image. It is a binary blob which U-Boot uses
> --

Regards,
Bin