From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Holler Date: Thu, 04 Nov 2010 13:47:43 +0100 Subject: [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support In-Reply-To: <4CD2A9A9.4010607@free.fr> References: <1288808663-15074-1-git-send-email-albert.aribaud@free.fr> <4CD28A3E.60406@ahsoftware.de> <4CD28EF0.5020506@ahsoftware.de> <4CD2A9A9.4010607@free.fr> Message-ID: <4CD2AB6F.2040507@ahsoftware.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 04.11.2010 13:40, schrieb Albert ARIBAUD: > Le 04/11/2010 11:46, Alexander Holler a ?crit : >> So the suggested change from Steve Sakoman (reordered fix from Albert >> Aribaud) still seems to be the one to prefer. > > Something is wrong here: > >> - .rel.dyn : { *(.rel.dyn) } >> + .rel.dyn : { *(.rel.*) } > > This change is the same as the one I submitted, replacing any .rel.dyn > input sections references with .rel* input sections (this includes > .rel.* sections), so I fail to see the difference. The difference is the place. Your patch v1 puts that after the end which results in a larger binary. This one just leaves the place as before and only changes what is included. In regard to your v3 I don't know what is the difference, I haven't looked at what it (v3) really does because I don't know much about the linker sections and (currently) don't want to drive deeper into that. > > Can you please perform a build and wrap the u-boot, u-boot.lds, > u-boot.map files and build standard and error outputs in a tarball and > send that to me? Will do so. Regards, Alexander