[PATCH] sysreset: psci: drop reliance on SPL support

[Andre Przywara]

On Tue, 23 Apr 2024 19:01:44 -0600
Tom Rini <trini at konsulko.com> wrote:

Hi Tom,

> 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.przywara at 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..

Ouch, you are of course right! Will send a fixed v2.

Cheers,
Andre

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