* [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