On Wed, Apr 24, 2024 at 12:46:04AM +0100, Andre Przywara wrote:
> At the moment enabling CONFIG_SYSRESET_PSCI *selects* SPL_ARM_PSCI_FW,
> even though this is a platform design property, so nothing any driver
> should enforce. Some platforms load the PSCI runtime (TF-A) only in the
> SPL, so PSCI is naturally not available during SPL runtime.
> 
> Spit CONFIG_SYSRESET_PSCI up into an SPL and a U-Boot proper version,
> and let the former *depend* on SPL_ARM_PSCI_FW.
> 
> This allows to enable CONFIG_SYSRESET_PSCI on platforms without SPL PSCI
> support.
> 
> Signed-off-by: Andre Przywara <andre.przyw...@arm.com>
> ---
>  drivers/sysreset/Kconfig  | 8 +++++++-
>  drivers/sysreset/Makefile | 2 +-
>  2 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/sysreset/Kconfig b/drivers/sysreset/Kconfig
> index 49c0787b26d..62fe9eb6220 100644
> --- a/drivers/sysreset/Kconfig
> +++ b/drivers/sysreset/Kconfig
> @@ -119,11 +119,17 @@ config SYSRESET_PALMAS
>  config SYSRESET_PSCI
>       bool "Enable support for PSCI System Reset"
>       depends on ARM_PSCI_FW
> -     select SPL_ARM_PSCI_FW if SPL
>       help
>         Enable PSCI SYSTEM_RESET function call.  To use this, PSCI firmware
>         must be running on your system.
>  
> +config SYSRESET_SPL_PSCI

This should be SPL_SYSRESET_PSCI so that..

[snip]
> -obj-$(CONFIG_SYSRESET_PSCI) += sysreset_psci.o
> +obj-$(CONFIG_$(SPL_TPL_)SYSRESET_PSCI) += sysreset_psci.o

This works.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to