u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: Handle .gnu.hash section in ldscripts
Date: Sat, 22 Feb 2014 14:27:53 +0100	[thread overview]
Message-ID: <20140222142753.6f0768b8@lilith> (raw)
In-Reply-To: <20140213123907.6b97040e@lilith>

Hi Albert,

On Thu, 13 Feb 2014 12:39:07 +0100, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:

> Hi Andreas,
> 
> On Mon, 27 Jan 2014 19:46:03 +0100, Andreas F?rber <afaerber@suse.de>
> wrote:
> 
> > Hello,
> > 
> > Am 27.01.2014 15:24, schrieb Wolfgang Denk:
> > > In message <1390798091-24747-1-git-send-email-afaerber@suse.de> you wrote:
> > >> Avoids "could not find output section .gnu.hash" ld.bfd errors on openSUSE.
> > >>
> > >> Cc: Albert Aribaud <albert.u.boot@aribaud.net>
> > >> Cc: Tom Rini <trini@ti.com>
> > >> Signed-off-by: Andreas F?rber <afaerber@suse.de>
> > >> ---
> > >>  More changes like this may be needed, but these I compile-tested with
> > >>  v2014.01 (regression from v2013.10). colibri_t20_iris was runtime-tested.
> > >>  GCC 4.8.2 r206511, binutils 2.24
> > >>
> > >>  arch/arm/cpu/u-boot.lds    | 1 +
> > >>  board/ti/am335x/u-boot.lds | 1 +
> > >>  2 files changed, 2 insertions(+)
> > > 
> > > Is this an ARM specific issue, or other architectures affected as
> > > well?
> > 
> > I figured out that the section was apparently not caught by the .gnu*
> > wildcard and this patch fixed our build for the following ARM configs:
> > 
> > am335x_evm
> > arndale
> > colibri_t20_iris
> > cubieboard
> > highbank
> > hyundai_a7hd
> > mele_a1000
> > mx53loco
> > omap3_beagle
> > omap4_panda
> > paz00
> > rpi_b
> >
> > My guess was that this could be related to the arm-only commit:
> > http://git.denx.de/?p=u-boot.git;a=commit;h=47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7
> > (which touches on a few more .lds files than mine)
> > 
> > But I see nothing ARM-specific in .gnu.hash sections, so other
> > architectures might in theory be affected as well.
> > Cf. http://sourceware.org/ml/binutils/2006-10/msg00377.html
> > 
> > I did read your post for ELDK pointing someone to an FAQ entry on not
> > using gcc-specific flags with ld, but for one I did not spot such a
> > thing in the ld command line, and for another our toolchain did not
> > change and v2013.10 builds succeed whereas v2014.01 builds failed
> > without the patch. I could try to git-bisect the offending change if
> > necessary.
> 
> Using gcc version 4.8.1 (Ubuntu/Linaro 4.8.1-10ubuntu7), I cannot see
> any issues on these boards:
> 
> am335x_evm
> colibri_t20_iris
> highbank
> mx53loco
> omap3_beagle
> omap4_panda
> paz00
> rpi_b
> 
> On arndale, issues are due to warnings unrelated to linking.
> 
> Cubieboard, hyundai_a7hd, mele_a1000 are not in mainline AFAICT (but I'd
> like to see mele_a1000 in, as this might mean mele_a1000g could
> follow. :) )
> 
> Can you point me to the toolchain you are using, and if you are not
> building mainline code, can you make this code available to me somehow?

Ping.

> > Best regards,
> > 
> > Andreas

Amicalement,
-- 
Albert.

  reply	other threads:[~2014-02-22 13:27 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-27  4:48 [U-Boot] [PATCH] arm: Handle .gnu.hash section in ldscripts Andreas Färber
2014-01-27 14:24 ` Wolfgang Denk
2014-01-27 18:46   ` Andreas Färber
2014-01-27 19:11     ` Wolfgang Denk
2014-01-28 14:01       ` Andreas Färber
2014-01-28 16:03         ` Wolfgang Denk
2014-02-13 11:39     ` Albert ARIBAUD
2014-02-22 13:27       ` Albert ARIBAUD [this message]
2014-03-02 17:57         ` Andreas Färber
2014-03-02 19:15           ` Albert ARIBAUD
2014-03-02 20:57             ` Andreas Färber
2014-03-02 22:00               ` Andreas Färber
2014-03-02 23:44                 ` Tom Rini
2014-03-03  6:58                 ` Albert ARIBAUD
2014-03-03  7:00                 ` Albert ARIBAUD
2014-03-09 20:05                   ` Albert ARIBAUD
2014-03-09 20:22                     ` Andreas Färber
2014-03-09 20:34                       ` Albert ARIBAUD
2014-03-09 20:52                         ` Andreas Färber
2014-03-10  7:24                           ` Albert ARIBAUD
2014-03-13 15:43                             ` Lukasz Majewski
2014-03-13 17:37                               ` Albert ARIBAUD
2014-03-14  7:37                                 ` Lukasz Majewski
2014-03-19  9:33                                   ` Albert ARIBAUD
2014-03-03  8:41                 ` Richard Biener
2014-03-03 13:25                   ` Tom Rini
2014-03-25  2:08 ` Simon Glass
2014-04-07  9:43 ` 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=20140222142753.6f0768b8@lilith \
    --to=albert.u.boot@aribaud.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).