On 02/03/2011 08:59 PM, daniel.schwierz...@googlemail.com wrote:
> The linker of recent toolchains complains about multiple definitions
> on final linking of u-boot binary. This patch removes all redundant
> object files from u-boot.lds those are already added to .text section
> by the linker.
Confirmed.
> That patch could not be tested but the resulting u-boot.map still looks
> good. The start symbol is at 0xB0000000, the environment at 0xB0008000
> so u-boot should boot.
Agreed.
> --- a/board/purple/u-boot.lds
> +++ b/board/purple/u-boot.lds
> @@ -36,11 +36,6 @@ SECTIONS
> {
> arch/mips/cpu/start.o (.text)
> board/purple/lowlevel_init.o (.text)
> - arch/mips/cpu/cache.o (.text)
> - common/main.o (.text)
> - common/dlmalloc.o (.text)
> - common/cmd_boot.o (.text)
> - lib/zlib.o (.text)
> . = DEFINED(env_offset) ? env_offset : .;
> common/env_embedded.o (.ppcenv)
This is ok, though we have some room for further cealnups. With having
.text statement in the lowlevel_init.S, we could remove all these lines.
Applied anyway, thanks for the patch.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
