[U-Boot] [PATCH 2/3] MIPS: Purple: Fix multiple definition error on final linking of u-boot binary
Wolfgang Denk
wd at denx.de
Sat Feb 5 15:15:12 CET 2011
Dear Shinya Kuribayashi,
In message <4D4D3E6B.2040605 at pobox.com> you wrote:
>
> > --- 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.
The prupose of explicitly pulling in these objects at that place is to
fill in the first flash sector below the sector(s) that is/are
reserved for the environment (we call this setup "embedded
environment").
But then, the Purple board has not been actively maintained for years,
and I guess we could as well remove it. At least I have not had
access to this board for several yars.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The more complex the mind, the greater the need for the simplicity of
play.
-- Kirk, "Shore Leave", stardate 3025.8
More information about the U-Boot
mailing list