From: "Álvaro Fernández Rojas" <noltari@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 2/4] u-boot.elf: allow overriding entry symbol
Date: Thu, 20 Apr 2017 20:36:26 +0200 [thread overview]
Message-ID: <1492713388-310-3-git-send-email-noltari@gmail.com> (raw)
In-Reply-To: <1492713388-310-1-git-send-email-noltari@gmail.com>
LD gives the following warning when trying to process u-boot-elf.o
warning: cannot find entry symbol __start; defaulting to 0000000080010000
According to gnu-libc the entry symbol for mips is __start and not _start:
https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/mips/dl-machine.h;h=ed47513ccc1d23d23d32ee640053d2f351f3990b;hb=HEAD#l258
Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
---
v4: Introduce changes suggested by Tom Rini:
- __start is the standard for MIPS, not ARM.
v3: Introduce changes suggested by Daniel Schwierzeck:
- Split patches.
v2: Introduce changes suggested by Daniel Schwierzeck:
- Fix _start vs __start symbol.
Makefile | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 8730550..6cf2568 100644
--- a/Makefile
+++ b/Makefile
@@ -1184,10 +1184,13 @@ u-boot-img-spl-at-end.bin: u-boot.img spl/u-boot-spl.bin FORCE
$(call if_changed,pad_cat)
# Create a new ELF from a raw binary file.
+ifndef PLATFORM_ELFENTRY
+ PLATFORM_ELFENTRY = "_start"
+endif
u-boot.elf: u-boot.bin
@$(OBJCOPY) -I binary $(PLATFORM_ELFFLAGS) $< u-boot-elf.o
@$(LD) u-boot-elf.o -o $@ \
- --defsym=_start=$(CONFIG_SYS_TEXT_BASE) \
+ --defsym=$(PLATFORM_ELFENTRY)=$(CONFIG_SYS_TEXT_BASE) \
-Ttext=$(CONFIG_SYS_TEXT_BASE)
# Rule to link u-boot
--
2.1.4
next prev parent reply other threads:[~2017-04-20 18:36 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-20 18:36 [U-Boot] [PATCH v4 0/4] u-boot.elf: support other archs Álvaro Fernández Rojas
2017-04-20 18:36 ` [U-Boot] [PATCH v4 1/4] u-boot.elf: remove hard-coded arm64 flags Álvaro Fernández Rojas
2017-04-30 18:41 ` Daniel Schwierzeck
2017-04-20 18:36 ` Álvaro Fernández Rojas [this message]
2017-04-20 19:57 ` [U-Boot] [PATCH v4 2/4] u-boot.elf: allow overriding entry symbol Tom Rini
2017-04-30 18:41 ` Daniel Schwierzeck
2017-04-20 18:36 ` [U-Boot] [PATCH v4 3/4] MIPS: add support for generating u-boot.elf Álvaro Fernández Rojas
2017-04-30 18:41 ` Daniel Schwierzeck
2017-04-20 18:36 ` [U-Boot] [PATCH v4 4/4] u-boot.elf: add quiet_cmd_u-boot-elf and cmd_u-boot-elf Álvaro Fernández Rojas
2017-04-20 20:06 ` Tom Rini
2017-04-30 18:41 ` Daniel Schwierzeck
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=1492713388-310-3-git-send-email-noltari@gmail.com \
--to=noltari@gmail.com \
--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