public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Alexander Graf <agraf@suse.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 03/11] efi_loader: Use compiler constants for image loader
Date: Thu, 14 Jun 2018 20:22:24 +0200	[thread overview]
Message-ID: <20180614182232.78201-4-agraf@suse.de> (raw)
In-Reply-To: <20180614182232.78201-1-agraf@suse.de>

The EFI image loader tries to determine which target architecture we're
working with to only load PE binaries that match.

So far this has worked based on CONFIG defines, because the target CPU
was always indicated by a config define. With sandbox however, this is
not longer true as all sandbox targets only encompass a single CONFIG
option and so we need to use compiler defines to determine the CPU
architecture.

Signed-off-by: Alexander Graf <agraf@suse.de>
---
 lib/efi_loader/efi_image_loader.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/lib/efi_loader/efi_image_loader.c b/lib/efi_loader/efi_image_loader.c
index ecdb77e5b6..fdf40a62c8 100644
--- a/lib/efi_loader/efi_image_loader.c
+++ b/lib/efi_loader/efi_image_loader.c
@@ -19,25 +19,25 @@ const efi_guid_t efi_simple_file_system_protocol_guid =
 const efi_guid_t efi_file_info_guid = EFI_FILE_INFO_GUID;
 
 static int machines[] = {
-#if defined(CONFIG_ARM64)
+#if defined(__aarch64__)
 	IMAGE_FILE_MACHINE_ARM64,
-#elif defined(CONFIG_ARM)
+#elif defined(__arm__)
 	IMAGE_FILE_MACHINE_ARM,
 	IMAGE_FILE_MACHINE_THUMB,
 	IMAGE_FILE_MACHINE_ARMNT,
 #endif
 
-#if defined(CONFIG_X86_64)
+#if defined(__x86_64__)
 	IMAGE_FILE_MACHINE_AMD64,
-#elif defined(CONFIG_X86)
+#elif defined(__i386__)
 	IMAGE_FILE_MACHINE_I386,
 #endif
 
-#if defined(CONFIG_CPU_RISCV_32)
+#if defined(__riscv) && (__riscv_xlen == 32)
 	IMAGE_FILE_MACHINE_RISCV32,
 #endif
 
-#if defined(CONFIG_CPU_RISCV_64)
+#if defined(__riscv) && (__riscv_xlen == 64)
 	IMAGE_FILE_MACHINE_RISCV64,
 #endif
 	0 };
-- 
2.12.3

  parent reply	other threads:[~2018-06-14 18:22 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-14 18:22 [U-Boot] [PATCH v2 00/11] sandbox: efi_loader support Alexander Graf
2018-06-14 18:22 ` [U-Boot] [PATCH v2 01/11] efi: sandbox: Add distroboot support Alexander Graf
2018-06-14 18:22 ` [U-Boot] [PATCH v2 02/11] efi: sandbox: Add relocation constants Alexander Graf
2018-06-14 18:22 ` Alexander Graf [this message]
2018-06-14 19:01   ` [U-Boot] [PATCH v2 03/11] efi_loader: Use compiler constants for image loader Simon Glass
2018-06-14 18:22 ` [U-Boot] [PATCH v2 04/11] efi_loader: Use map_sysmem() in bootefi command Alexander Graf
2018-06-14 19:00   ` Simon Glass
2018-06-14 18:22 ` [U-Boot] [PATCH v2 05/11] efi.h: Do not use config options Alexander Graf
2018-06-14 18:22 ` [U-Boot] [PATCH v2 06/11] efi_loader: Allow SMBIOS tables in highmem Alexander Graf
2018-06-14 19:01   ` Simon Glass
2018-06-14 19:13     ` Alexander Graf
2018-06-14 19:26       ` Heinrich Schuchardt
2018-06-14 19:34         ` Alexander Graf
2018-06-14 18:22 ` [U-Boot] [PATCH v2 07/11] sandbox: Map host memory for efi_loader Alexander Graf
2018-06-14 19:02   ` Simon Glass
2018-06-14 19:15     ` Alexander Graf
2018-06-14 21:36       ` Simon Glass
2018-06-14 19:21     ` Heinrich Schuchardt
2018-06-14 21:35       ` Simon Glass
2018-06-14 18:22 ` [U-Boot] [PATCH v2 08/11] efi_loader: efi_allocate_pages is too restrictive Alexander Graf
2018-06-14 18:22 ` [U-Boot] [PATCH v2 09/11] efi_loader: Disable miniapps on sandbox Alexander Graf
2018-06-14 19:01   ` Simon Glass
2018-06-14 18:22 ` [U-Boot] [PATCH v2 10/11] efi_loader: Pass address to fs_read() Alexander Graf
2018-06-14 19:01   ` Simon Glass
2018-06-14 19:51     ` Alexander Graf
2018-06-14 21:35       ` Simon Glass
2018-06-14 21:55         ` Alexander Graf
2018-06-19 22:03           ` Simon Glass
2018-06-20  9:31             ` Alexander Graf
2018-06-21  2:02               ` Simon Glass
2018-06-14 18:22 ` [U-Boot] [PATCH v2 11/11] efi: sandbox: Enable EFI loader for sandbox Alexander Graf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180614182232.78201-4-agraf@suse.de \
    --to=agraf@suse.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox