From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add gc-section support for ARM
Date: Thu, 17 May 2012 11:53:12 -0700 [thread overview]
Message-ID: <20120517185312.GC23562@bill-the-cat> (raw)
In-Reply-To: <201205161045.33983.manningc2@actrix.gen.nz>
On Wed, May 16, 2012 at 10:45:33AM +1200, Charles Manning wrote:
> 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?
Yes, MAKEALL in the top level. Assuming you've installed ELDK5.1 or 4.2
to their default locations:
$ source /opt/eldk-5.1/armv7a/environment-setup-armv7a-vfp-neon-linux-gnueabi
$ CROSS_COMPILE=arm-linux-gnueabi- BUILD_DIR=eldk51 ./MAKEALL -a arm
$ CROSS_COMPILE=/opt/eldk-4.2/arm/usr/bin/arm-linux-gnueabi- BUILD_DIR=eldk42 ./MAKEALL -a arm
(I've got the above in a script with a few tweaks to build all TI stuff,
for example).
--
Tom
next prev parent reply other threads:[~2012-05-17 18:53 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
2012-05-17 18:53 ` Tom Rini [this message]
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=20120517185312.GC23562@bill-the-cat \
--to=trini@ti.com \
--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.