From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Sat, 05 Feb 2011 15:15:12 +0100 Subject: [U-Boot] [PATCH 2/3] MIPS: Purple: Fix multiple definition error on final linking of u-boot binary In-Reply-To: <4D4D3E6B.2040605@pobox.com> References: <1296734392-15203-1-git-send-email-daniel.schwierzeck@googlemail.com> <1296734392-15203-3-git-send-email-daniel.schwierzeck@googlemail.com> <4D4D3E6B.2040605@pobox.com> Message-ID: <20110205141512.8DFBD15242F@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Shinya Kuribayashi, In message <4D4D3E6B.2040605@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