All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support
Date: Thu, 04 Nov 2010 13:47:43 +0100	[thread overview]
Message-ID: <4CD2AB6F.2040507@ahsoftware.de> (raw)
In-Reply-To: <4CD2A9A9.4010607@free.fr>

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

  reply	other threads:[~2010-11-04 12:47 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-03 18:24 [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support Albert Aribaud
2010-11-03 18:24 ` [U-Boot] [PATCH V3 2/3] tx25: " Albert Aribaud
2010-11-03 18:24 ` [U-Boot] [PATCH V3 3/3] tx25: increase u-boot NAND size to match current build sizes Albert Aribaud
2010-11-03 18:27 ` [U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support Albert ARIBAUD
2010-11-04 10:26 ` Alexander Holler
2010-11-04 10:46   ` Alexander Holler
2010-11-04 12:40     ` Albert ARIBAUD
2010-11-04 12:47       ` Alexander Holler [this message]
2010-11-04 12:55         ` Albert ARIBAUD
2010-11-04 13:18           ` Alexander Holler
2010-11-04 16:56             ` Albert ARIBAUD
2010-11-04 18:37               ` Alexander Holler
2010-11-04 18:42                 ` Alexander Holler
2010-11-04 19:20                   ` Albert ARIBAUD
2010-11-04 19:39                     ` Alexander Holler
2010-11-04 22:06                       ` Albert ARIBAUD

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4CD2AB6F.2040507@ahsoftware.de \
    --to=holler@ahsoftware.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.