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

2024-05-18 Thread Heinrich Schuchardt

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





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

2024-05-17 Thread Jiaxun Yang
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 
---
 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) || \
+   (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

-- 
2.34.1