mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] ARM: psci: hide ARM_PSCI options when ARCH_IMX7 is disabled
@ 2024-09-11  8:17 Ahmad Fatoum
  2024-09-11  8:30 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2024-09-11  8:17 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

We only support providing PSCI on a single SoC, the i.MX7. Having the
option in other configuration is thus confusing.

Therefore hide the symbols when not building for the i.MX7. Using PSCI
as consumer is a separate symbol (ARM_PSCI_CLIENT) and should be
unaffected.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 arch/arm/Kconfig          | 11 +++++++++--
 arch/arm/mach-imx/Kconfig |  1 +
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 6406b8ebe073..6815e4de46bd 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -412,9 +412,16 @@ config ARM_SECURE_MONITOR
 config ARM_PSCI_OF
 	bool
 
+config HAS_PSCI_PROVIDER
+	bool
+	help
+	  Selected by platforms that implement PSCI inside barebox.
+	  This is uncommon as barebox is usually PSCI client on systems
+	  with PSCI with the provider being ARM Trusted Firmware.
+
 config ARM_PSCI
-	bool "enable Power State Coordination Interface (PSCI) support"
-	depends on CPU_V7
+	bool "Provide Power State Coordination Interface (PSCI) support"
+	depends on HAS_PSCI_PROVIDER
 	select ARM_SECURE_MONITOR
 	select ARM_PSCI_OF
 	help
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index 96ab72315732..d1f8dc795b57 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -143,6 +143,7 @@ config ARCH_IMX6UL
 config ARCH_IMX7
 	bool
 	select CPU_V7
+	select HAS_PSCI_PROVIDER
 	select PINCTRL_IMX_IOMUX_V3
 	select COMMON_CLK_OF_PROVIDER
 	select ARCH_HAS_FEC_IMX
-- 
2.39.2




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] ARM: psci: hide ARM_PSCI options when ARCH_IMX7 is disabled
  2024-09-11  8:17 [PATCH] ARM: psci: hide ARM_PSCI options when ARCH_IMX7 is disabled Ahmad Fatoum
@ 2024-09-11  8:30 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2024-09-11  8:30 UTC (permalink / raw)
  To: barebox, Ahmad Fatoum


On Wed, 11 Sep 2024 10:17:30 +0200, Ahmad Fatoum wrote:
> We only support providing PSCI on a single SoC, the i.MX7. Having the
> option in other configuration is thus confusing.
> 
> Therefore hide the symbols when not building for the i.MX7. Using PSCI
> as consumer is a separate symbol (ARM_PSCI_CLIENT) and should be
> unaffected.
> 
> [...]

Applied, thanks!

[1/1] ARM: psci: hide ARM_PSCI options when ARCH_IMX7 is disabled
      https://git.pengutronix.de/cgit/barebox/commit/?id=1a41e8ab8915 (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-09-11  8:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-09-11  8:17 [PATCH] ARM: psci: hide ARM_PSCI options when ARCH_IMX7 is disabled Ahmad Fatoum
2024-09-11  8:30 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox