[U-Boot] [PATCH 1/4] tegra2: Avoid warnings if CONFIG_SYS_TEXT_BASE is redefined.

[Tom Warren]

Thierry/Stephen,

> -----Original Message-----
> From: Thierry Reding [mailto:thierry.reding at avionic-design.de]
> Sent: Monday, November 14, 2011 10:23 AM
> To: Simon Glass
> Cc: Stephen Warren; Tom Warren; u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH 1/4] tegra2: Avoid warnings if
> CONFIG_SYS_TEXT_BASE is redefined.
> 
> * PGP Signed by an unknown key
> 
> * Simon Glass wrote:
> > Hi Stephen,
> >
> > On Mon, Nov 14, 2011 at 9:09 AM, Stephen Warren <swarren at nvidia.com>
> wrote:
> > > Thierry Reding wrote at Monday, November 14, 2011 6:22 AM:
> > >> If a board configuration file redefines CONFIG_SYS_TEXT_BASE, the U-
> Boot
> > >> build system will usually define it explicitly on the compiler command-
> > >> line, which will cause the define in tegra2-common.h to emit a
> > >> redefinition warning.
> > >>
> > >> To allow boards to redefine CONFIG_SYS_TEXT_BASE, tegra2-common.h now
> > >> only defines CONFIG_SYS_TEXT_BASE when it isn't already defined by the
> > >> board configuration or the command-line respectively.
> > >
> > > Instead of doing this, wouldn't it make sense to just switch all Tegra
> > > boards to TEXT_BASE=00108000 to match NVIDIA's flashing tool's
> assumptions?
> > > That way, no board would need to override TEXT_BASE at all.
> > >
> > > These assumptions are true for all boards in the standard flashing tool
> > > builds; it's just that non-standard builds exist for some boards (those
> > > that happened to be upstreamed to U-Boot first) that allow use of
> 00e08000
> > > instead.
> > >
> > > Tom, Simon, what are your thoughts here?
> >
[Tom Warren] Note that when T30 (Tegra3/Kal-el) hits upstream U-Boot its TEXT_BASE will be 0x80108000/0x80E08000 since T30's SDRAM starts @ 2GB. What'll happen then?

> > I was going to say the same thing.
> 
> Okay, I can integrate that into the second version. Or would you prefer a
> separate patch?
> 
> Thierry
> 
> * Unknown Key
> * 0xA2E3269F
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------