[U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

[Tom Warren]

Albert,


On Thu, Apr 11, 2013 at 10:59 AM, Albert ARIBAUD
<albert.u.boot at aribaud.net>wrote:

> Hi Tom,
>
> On Thu, 11 Apr 2013 10:52:08 -0700, Tom Warren <TWarren at nvidia.com>
> wrote:
>
> > Albert,
> >
> > > -----Original Message-----
> > > From: Stephen Warren [mailto:swarren at wwwdotorg.org]
> > > Sent: Friday, April 05, 2013 8:59 AM
> > > To: Tom Rini
> > > Cc: Albert ARIBAUD; Scott Wood; u-boot at lists.denx.de;
> > > Stephen at theia.denx.de; Tom Warren; Simon Glass; Allen Martin
> > > Subject: Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in
> linker
> > > scripts
> >
> > I just fetched & merged TOT u-boot-arm (getting ready for a PR from
> u-boot-tegra -> u-boot-arm), and did a test build of all Tegra boards.
> >
> > I get the following linker (ld) error on all builds due to this commit
> (abbecf4c8,  ARM: Fix __bss_start and __bss_end in linker scripts):
> >
> >  arm-linux-gnueabi-ld.bfd:u-boot.lds:44: syntax error
> >
> > This is pointing to the 'HIDDEN(__bss_base = .);' lines added in this
> change (in the generated u-boot.lds in the root dir).
> >
> > I'm seeing this with both gcc 4.4.1/ld 2.19.51  and gcc 4.6.2/ld 2.22
> tools.
> >
> > What compiler/binutils did you use to build/test this?
>
> This has been detected already, and one of the reasons why Tom and I
> decided to roll back ARM. Normally, you should have u-boot-arm/master =
> u-boot-arm/master = dd2445ec1b839a5ca61ff8438a5b7aebb21b7986.
>

>you should have u-boot-arm/master = u-boot-arm/master =
dd2445ec1b839a5ca61ff8438a5b7aebb21b7986.

I would hope that u-boot-arm/master would always == u-boot-arm/master ;)

That commit is not 5 patches back (from our other thread 'Rollback *again*
-- 5 patches), but more like 45.

I'll wait to do a fetch/rebase/PR for u-boot-tegra until the Tegra builds
are fixed.

Thanks,

Tom

>
> > Tom
>
> Amicalement,
> --
> Albert.
>