On Tue, Apr 18, 2017 at 10:35:32PM +0200, Álvaro Fernández Rojas wrote:

> LD gives the following warning when trying to process u-boot-elf.o
> warning: cannot find entry symbol __start; defaulting to 0000000080010000
> Fix this by renaming _start to __start
> 
> Signed-off-by: Álvaro Fernández Rojas <nolt...@gmail.com>
> ---
>  v3: Introduce changes suggested by Daniel Schwierzeck:
>   - Split patches.
>  v2: Introduce changes suggested by Daniel Schwierzeck:
>   - Fix _start vs __start symbol.
> 
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Makefile b/Makefile
> index 8730550..66c9be2 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1187,7 +1187,7 @@ u-boot-img-spl-at-end.bin: u-boot.img 
> spl/u-boot-spl.bin FORCE
>  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=__start=$(CONFIG_SYS_TEXT_BASE) \
>               -Ttext=$(CONFIG_SYS_TEXT_BASE)
>  
>  # Rule to link u-boot

I don't know what's going on here, but I don't think this is right.  We
don't see this warning on aarch64 today.  I would think that since we're
talking about ELF _start vs __start would be part of the standard.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to