public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Charles Manning <manningc2@actrix.gen.nz>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add gc-section support for ARM
Date: Wed, 16 May 2012 10:45:33 +1200	[thread overview]
Message-ID: <201205161045.33983.manningc2@actrix.gen.nz> (raw)
In-Reply-To: <20120515222526.B26B520436D@gemini.denx.de>

On Wednesday 16 May 2012 10:25:26 Wolfgang Denk wrote:
> Dear Charles,
>
> In message <201205161007.59560.manningc2@actrix.gen.nz> you wrote:
> > On Tuesday 15 May 2012 17:12:05 Mike Frysinger wrote:
>
> ...
>
> > > current u-boot policy is to not introduce changes that knowingly break
> > > other platforms.  so if you want to update the common arm config.mk,
> > > you need to at least compile test all arm boards.
>
> ---------------------^^^^^^^^^^^^
>
> > Doesn't that makes it an impossible task? Not having all the boards makes
> > it pretty hard to test them all.
>
> A compile test does not require any actual hardware.

Does the patch fail I submitted fail any compile tests?

AFAIK, it compiles but the resulting binary might just be rubbish on some 
boards.

It worked fine on my overo board without any changes to the overo lds.

I have just verified that it builds on
overo
mx51evk
davinci_sonata

If there is an expectation to build every possible config is there a script 
that will do that?

>
> > > other arches are using both fine, so it isn't a problem of common code.
> >
> > I might be incorrect, but I have noticed what appears to be some
> > differences between different versions of binutils.
>
> Please be more specific. The same "different versions of binutils"
> appear to work fine on other architectures ?

That was based on some observations a year or two ago. I don't know if the 
problem has now gone away and have no way of reproducing it.

-- Charles

  reply	other threads:[~2012-05-15 22:45 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-03  1:37 [U-Boot] [PATCH] Add gc-section support for ARM Charles Manning
2012-05-14  5:15 ` Mike Frysinger
2012-05-14 21:01   ` Charles Manning
2012-05-15  5:12     ` Mike Frysinger
2012-05-15  7:01       ` Albert ARIBAUD
2012-05-15 22:07       ` Charles Manning
2012-05-15 22:25         ` Wolfgang Denk
2012-05-15 22:45           ` Charles Manning [this message]
2012-05-17 18:53             ` Tom Rini
2012-05-29 21:03     ` Wolfgang Denk

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=201205161045.33983.manningc2@actrix.gen.nz \
    --to=manningc2@actrix.gen.nz \
    --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