Partially revert "efi_loader: Rename sections to allow for implicit data"

This partially reverts commit 7e21fbca26d18327cf7cabaad08df276a06a07d8.

That change broke sandbox EFI support for unknown reasons. It also changes
sandbox to use--gc-sections which we don't want.

For now I am just reverting the sandbox portion as presumably this change
is safe on other architectures.

Fixes: 7e21fbca26 (efi_loader: Rename sections to allow for implicit data)
Signed-off-by: Simon Glass <sjg@chromium.org>

diff --git a/arch/sandbox/config.mk b/arch/sandbox/config.mk
index 5e7077bfe75da80ecc3f0ee901a8e963c7843789..2babcde8815d3987ea4ea6c50fdb4448a61aeba6 100644 (file)
--- a/arch/sandbox/config.mk
+++ b/arch/sandbox/config.mk
@@ -5,9 +5,6 @@ PLATFORM_CPPFLAGS += -D__SANDBOX__ -U_FORTIFY_SOURCE
 PLATFORM_CPPFLAGS += -DCONFIG_ARCH_MAP_SYSMEM
 PLATFORM_LIBS += -lrt
 
-LDFLAGS_FINAL += --gc-sections
-PLATFORM_RELFLAGS += -ffunction-sections -fdata-sections
-
 # Define this to avoid linking with SDL, which requires SDL libraries
 # This can solve 'sdl-config: Command not found' errors
 ifneq ($(NO_SDL),)

diff --git a/arch/sandbox/cpu/u-boot.lds b/arch/sandbox/cpu/u-boot.lds
index 40c22143018e509e9fd9738e4d89e14fb9d93687..6a26c27e8e2d1d253e50150b072f9f1e43e702d8 100644 (file)
--- a/arch/sandbox/cpu/u-boot.lds
+++ b/arch/sandbox/cpu/u-boot.lds
@@ -22,9 +22,8 @@ SECTIONS
        }
 
        .efi_runtime : {
-               *(.text.efi_runtime*)
-               *(.rodata.efi_runtime*)
-               *(.data.efi_runtime*)
+               *(efi_runtime_text)
+               *(efi_runtime_data)
        }
 
        .__efi_runtime_stop : {
@@ -37,8 +36,8 @@ SECTIONS
        }
 
        .efi_runtime_rel : {
-               *(.rel*.efi_runtime)
-               *(.rel*.efi_runtime.*)
+               *(.relefi_runtime_text)
+               *(.relefi_runtime_data)
        }
 
        .efi_runtime_rel_stop :