diff mbox series

[2/3] EFI: Make EFI loader depend sandbox HOST_ARCH

Message ID 20240517-sandbox-hostarch-kconfig-v1-2-716c146635c9@flygoat.com
State Superseded
Delegated to: Tom Rini
Headers show
Series sandbox: HOST_ARCH improvements | expand

Commit Message

Jiaxun Yang May 17, 2024, 10:09 p.m. UTC
Since EFI loader on sandbox requires some architecture
code, make it depend on HOST_ARCH to ensure it's only
compiled on desired host.

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
---
 lib/efi_loader/Kconfig | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

Comments

Heinrich Schuchardt May 18, 2024, 8:15 p.m. UTC | #1
On 5/18/24 00:09, Jiaxun Yang wrote:
> Since EFI loader on sandbox requires some architecture
> code, make it depend on HOST_ARCH to ensure it's only
> compiled on desired host.
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>   lib/efi_loader/Kconfig | 7 +++++--
>   1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> index 430bb7f0f7dc..54d180fe31c1 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -1,11 +1,14 @@
>   config EFI_LOADER
>   	bool "Support running UEFI applications"
> -	depends on OF_LIBFDT && ( \
> +	depends on OF_LIBFDT && (( \
>   		ARM && (SYS_CPU = arm1136 || \
>   			SYS_CPU = arm1176 || \
>   			SYS_CPU = armv7   || \
>   			SYS_CPU = armv8)  || \
> -		X86 || RISCV || SANDBOX)
> +		X86 || RISCV) || \

The newly parentheses are not needed.

> +		(SANDBOX && (HOST_ARCH_X86 || HOST_ARCH_X86_64 || \
> +			     HOST_ARCH_ARM || HOST_ARCH_AARCH64 || \
> +			     HOST_ARCH_RISCV32 || HOST_ARCH_RISCV64)))
>   	# We need EFI_STUB_64BIT to be set on x86_64 with EFI_STUB
>   	depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT
>   	# We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB
>
diff mbox series

Patch

diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
index 430bb7f0f7dc..54d180fe31c1 100644
--- a/lib/efi_loader/Kconfig
+++ b/lib/efi_loader/Kconfig
@@ -1,11 +1,14 @@ 
 config EFI_LOADER
 	bool "Support running UEFI applications"
-	depends on OF_LIBFDT && ( \
+	depends on OF_LIBFDT && (( \
 		ARM && (SYS_CPU = arm1136 || \
 			SYS_CPU = arm1176 || \
 			SYS_CPU = armv7   || \
 			SYS_CPU = armv8)  || \
-		X86 || RISCV || SANDBOX)
+		X86 || RISCV) || \
+		(SANDBOX && (HOST_ARCH_X86 || HOST_ARCH_X86_64 || \
+			     HOST_ARCH_ARM || HOST_ARCH_AARCH64 || \
+			     HOST_ARCH_RISCV32 || HOST_ARCH_RISCV64)))
 	# We need EFI_STUB_64BIT to be set on x86_64 with EFI_STUB
 	depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT
 	# We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB