[U-Boot] [PATCH v6 07/13] efi: Add a comment about duplicated ELF constants
Simon Glass
sjg at chromium.org
Wed Jun 13 02:37:22 UTC 2018
These constants are defined in arch-specific code but redefined here. Add
a TODO to clean this up.
Signed-off-by: Simon Glass <sjg at chromium.org>
Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
---
Changes in v6: None
Changes in v5:
- Drop period after "elf" in comment
Changes in v4: None
Changes in v3: None
Changes in v2: None
lib/efi_loader/efi_runtime.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/lib/efi_loader/efi_runtime.c b/lib/efi_loader/efi_runtime.c
index acda21c91d..388dfb9840 100644
--- a/lib/efi_loader/efi_runtime.c
+++ b/lib/efi_loader/efi_runtime.c
@@ -28,6 +28,10 @@ static efi_status_t __efi_runtime EFIAPI efi_unimplemented(void);
static efi_status_t __efi_runtime EFIAPI efi_device_error(void);
static efi_status_t __efi_runtime EFIAPI efi_invalid_parameter(void);
+/*
+ * TODO(sjg at chromium.org): These defines and structs should come from the elf
+ * header for each arch (or a generic header) rather than being repeated here.
+ */
#if defined(CONFIG_ARM64)
#define R_RELATIVE 1027
#define R_MASK 0xffffffffULL
--
2.18.0.rc1.244.gcf134e6275-goog
More information about the U-Boot
mailing list