Re: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-23 Thread Tom Warren
Tegra devs,

Since I've not seen any objections (or discussion, really, beyond Simon/Allen), 
I'm going to apply u-boot-tegra/next to tegra/master (i.e. the SPL patches will 
now be de rigueur for u-boot-tegra going forward).

If you have Tegra patches that you are working on, or are in the middle of 
review/revision on the list (Simon's LCD and NAND), then you'll need to 
rework/rebase them on top of u-boot-tegra/master (w/the SPL changes) from this 
point on.

Thanks,

Tom

> -Original Message-
> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass
> Sent: Thursday, July 19, 2012 1:09 PM
> To: Tom Warren
> Cc: Allen Martin; swar...@wwwdotorg.org; thierry.red...@avionic-design.de;
> u-boot@lists.denx.de; Igor Grinberg; Konstantin Sinyuk
> Subject: Re: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
> 
> Hi Tom / Allen,
> 
> On Thu, Jul 19, 2012 at 4:37 PM, Tom Warren  wrote:
> > Allen,
> >
> >> -Original Message-
> >> From: Allen Martin [mailto:amar...@nvidia.com]
> >> Sent: Wednesday, July 18, 2012 5:02 PM
> >> To: Tom Warren
> >> Cc: swar...@wwwdotorg.org; s...@chromium.org; thierry.reding@avionic-
> >> design.de; u-boot@lists.denx.de
> >> Subject: Re: [PATCH v7 00/15] split tegra20 arm7 code into separate
> >> SPL
> >>
> >> On Tue, Jul 17, 2012 at 12:32:53PM -0700, Tom Warren wrote:
> >> > Allen,
> >> >
> >> > > -Original Message-
> >> > > From: Allen Martin [mailto:amar...@nvidia.com]
> >> > > Sent: Monday, July 16, 2012 4:02 PM
> >> > > To: Tom Warren; swar...@wwwdotorg.org; s...@chromium.org;
> >> > > thierry.red...@avionic-design.de
> >> > > Cc: u-boot@lists.denx.de; Allen Martin
> >> > > Subject: [PATCH v7 00/15] split tegra20 arm7 code into separate
> >> > > SPL
> >> > >
> >> > > This patch series fixes a long standing problem with the tegra20
> >> > > u-boot build.  Tegra20 contains an ARM7TDMI boot processor and a
> >> > > Cortex A9 main processor.  Prior to this patch series this was
> >> > > accomplished by #ifdefing out any armv7 code from the early boot
> >> > > sequence and creating a single binary that runs on both both the
> >> > > ARM7TDMI and A9.  This was very fragile as changes to compiler
> >> > > options or any additions or rearranging of the early boot code
> >> > > could add additional armv7 specific code causing it to fail on the
> ARM7TDMI.
> >> > >
> >> > > This patch series pulls all the armv4t code out into a separate
> >> > > SPL that does nothing more than initialize the A9 and transfer
> >> > > control to it.  The resultint SPL and armv7 u-boot are
> >> > > concatenated together into a single image.
> >> > >
> >> > > This patch series is also available from:
> >> > > git://github.com/arm000/u-boot.git
> >> > > branch: tegra-spl-v7
> >> > >
> >> >
> >> > Applied to u-boot-tegra/next AOK, tested on my Seaboard AOK, so:
> >> > Tested-by: Tom Warren 
> >> >
> >> > Note that I was confused by the final binary name
> >> > (u-boot-dtb-tegra.bin),
> >> since I'm used to flashing u-boot-dtb.bin.
> >> >
> >> > We need to come to a consensus about the final binary name for
> >> > Tegra U-
> >> Boot (I'd thought we had, and that it would be u-boot-dtb.bin, since
> >> that's what most devs are used to looking for in Tegra builds).
> 
> I think so.
> 
> >> >
> >>
> >> Yeah, I'd like some stability there too.  The -dtb rule is not tegra
> >> specific, which is why I didn't want to modify or remove it.  I think
> >> we're the only one that uses it though, so maybe it's not so bad.
> 
> Not for long :-)
> 
> >>
> >> > Also, one nit: I see the 2 sign-on strings (U-Boot SPL 2012.04.xxx,
> >> > and
> >> then U-Boot 2012.04.xxx), separated by 2 lines. I think it'd look
> >> better if you had them one right after the other, i.e. eliminate the
> extra linefeeds.
> >> >
> 
> >>
> >> The extra lines come from display_banner() which is ARM generic from
> >> the main u-boot.  I assume they are there to separate the banner from
> >> any junk that was on your screen before you rebooted, so it would
> >> make sense to move them to the SPL banner instead if you have SPL
> enabled.
> >> I'll make a separate patch for that in a week after I get back from
> >> vacation.
> 
> I suspect you could remove the extra line by not printing a \n in SPL.
> The other one might be a bit tricky as I think it is in U-Boot proper as you
> say.
> 
> Also do we need the full version tag on the SPL version?
> 
> >>
> >> -Allen
> >
> > Cool, thanks. Until then:
> >
> > Tegra2 SPL patches have been applied to u-boot-tegra/next & pushed to
> Denx. I'm going to hold off putting it into tegra/master and generating a
> pull request for awhile to allow all Tegra devs to test it, comment, etc.,
> since it's a major change to the Tegra build.
> >
> > If possible, please post any new Tegra changes against tegra/next (i.e.
> using Allen's SPL file locations).
> 
> I have a few minor comments on the series now that I have made time to go
> through it in final form:
> 
> 1. In the r

Re: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-19 Thread Simon Glass
Hi Tom / Allen,

On Thu, Jul 19, 2012 at 4:37 PM, Tom Warren  wrote:
> Allen,
>
>> -Original Message-
>> From: Allen Martin [mailto:amar...@nvidia.com]
>> Sent: Wednesday, July 18, 2012 5:02 PM
>> To: Tom Warren
>> Cc: swar...@wwwdotorg.org; s...@chromium.org; thierry.reding@avionic-
>> design.de; u-boot@lists.denx.de
>> Subject: Re: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
>>
>> On Tue, Jul 17, 2012 at 12:32:53PM -0700, Tom Warren wrote:
>> > Allen,
>> >
>> > > -Original Message-
>> > > From: Allen Martin [mailto:amar...@nvidia.com]
>> > > Sent: Monday, July 16, 2012 4:02 PM
>> > > To: Tom Warren; swar...@wwwdotorg.org; s...@chromium.org;
>> > > thierry.red...@avionic-design.de
>> > > Cc: u-boot@lists.denx.de; Allen Martin
>> > > Subject: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
>> > >
>> > > This patch series fixes a long standing problem with the tegra20
>> > > u-boot build.  Tegra20 contains an ARM7TDMI boot processor and a
>> > > Cortex A9 main processor.  Prior to this patch series this was
>> > > accomplished by #ifdefing out any armv7 code from the early boot
>> > > sequence and creating a single binary that runs on both both the
>> > > ARM7TDMI and A9.  This was very fragile as changes to compiler
>> > > options or any additions or rearranging of the early boot code could
>> > > add additional armv7 specific code causing it to fail on the ARM7TDMI.
>> > >
>> > > This patch series pulls all the armv4t code out into a separate SPL
>> > > that does nothing more than initialize the A9 and transfer control
>> > > to it.  The resultint SPL and armv7 u-boot are concatenated together
>> > > into a single image.
>> > >
>> > > This patch series is also available from:
>> > > git://github.com/arm000/u-boot.git
>> > > branch: tegra-spl-v7
>> > >
>> >
>> > Applied to u-boot-tegra/next AOK, tested on my Seaboard AOK, so:
>> > Tested-by: Tom Warren 
>> >
>> > Note that I was confused by the final binary name (u-boot-dtb-tegra.bin),
>> since I'm used to flashing u-boot-dtb.bin.
>> >
>> > We need to come to a consensus about the final binary name for Tegra U-
>> Boot (I'd thought we had, and that it would be u-boot-dtb.bin, since that's
>> what most devs are used to looking for in Tegra builds).

I think so.

>> >
>>
>> Yeah, I'd like some stability there too.  The -dtb rule is not tegra
>> specific, which is why I didn't want to modify or remove it.  I think we're
>> the only one that uses it though, so maybe it's not so bad.

Not for long :-)

>>
>> > Also, one nit: I see the 2 sign-on strings (U-Boot SPL 2012.04.xxx, and
>> then U-Boot 2012.04.xxx), separated by 2 lines. I think it'd look better if
>> you had them one right after the other, i.e. eliminate the extra linefeeds.
>> >

>>
>> The extra lines come from display_banner() which is ARM generic from the
>> main u-boot.  I assume they are there to separate the banner from any junk
>> that was on your screen before you rebooted, so it would make sense to move
>> them to the SPL banner instead if you have SPL enabled.
>> I'll make a separate patch for that in a week after I get back from
>> vacation.

I suspect you could remove the extra line by not printing a \n in SPL.
The other one might be a bit tricky as I think it is in U-Boot proper
as you say.

Also do we need the full version tag on the SPL version?

>>
>> -Allen
>
> Cool, thanks. Until then:
>
> Tegra2 SPL patches have been applied to u-boot-tegra/next & pushed to Denx. 
> I'm going to hold off putting it into tegra/master and generating a pull 
> request for awhile to allow all Tegra devs to test it, comment, etc., since 
> it's a major change to the Tegra build.
>
> If possible, please post any new Tegra changes against tegra/next (i.e. using 
> Allen's SPL file locations).

I have a few minor comments on the series now that I have made time to
go through it in final form:

1. In the resulting -tegra.bin image I see this:

00108038 <_fiq>:
  108038:   00108038.word   0x00108038
  10803c:   deadbeef.word   0xdeadbeef

00108040 <_TEXT_BASE>:
  108040:   0010c000.word   0x0010c000

To me it seems odd that SPL shows a TEXT_BASE of 10c000 when we
actually need to load it at 108000. Can we change that? Also due to
the difference between arm720 and armv7 there is no 0x12345678 tag
before the text base. It would be nice if we could have that, as it is
a convenient tag to point to the text base.

In the 'ARM: add tegra20 support' patch:

2. lastdec = 0 seems unnecessary since it should already be 0 at init.

3. Can we init the JTAG earlier (before any serial output, for
example)? It may be useful to be able to set breakpoints in SPL.

In 'tegra20: add u-boot-*-tegra.bin targets':

4. The Makefile stuff could perhaps be split out a bit. You have:

ifeq ($(SOC),tegra20)
ifeq ($(CONFIG_OF_SEPARATE),y)
$(obj)u-boot-dtb-tegra.bin: $(obj)spl/u-boot-spl.bin $(obj)u-boot.bin
$(obj)u-boot.dtb
$(OBJCOPY) ${OB

Re: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-19 Thread Tom Warren
Allen,

> -Original Message-
> From: Allen Martin [mailto:amar...@nvidia.com]
> Sent: Wednesday, July 18, 2012 5:02 PM
> To: Tom Warren
> Cc: swar...@wwwdotorg.org; s...@chromium.org; thierry.reding@avionic-
> design.de; u-boot@lists.denx.de
> Subject: Re: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
> 
> On Tue, Jul 17, 2012 at 12:32:53PM -0700, Tom Warren wrote:
> > Allen,
> >
> > > -Original Message-
> > > From: Allen Martin [mailto:amar...@nvidia.com]
> > > Sent: Monday, July 16, 2012 4:02 PM
> > > To: Tom Warren; swar...@wwwdotorg.org; s...@chromium.org;
> > > thierry.red...@avionic-design.de
> > > Cc: u-boot@lists.denx.de; Allen Martin
> > > Subject: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
> > >
> > > This patch series fixes a long standing problem with the tegra20
> > > u-boot build.  Tegra20 contains an ARM7TDMI boot processor and a
> > > Cortex A9 main processor.  Prior to this patch series this was
> > > accomplished by #ifdefing out any armv7 code from the early boot
> > > sequence and creating a single binary that runs on both both the
> > > ARM7TDMI and A9.  This was very fragile as changes to compiler
> > > options or any additions or rearranging of the early boot code could
> > > add additional armv7 specific code causing it to fail on the ARM7TDMI.
> > >
> > > This patch series pulls all the armv4t code out into a separate SPL
> > > that does nothing more than initialize the A9 and transfer control
> > > to it.  The resultint SPL and armv7 u-boot are concatenated together
> > > into a single image.
> > >
> > > This patch series is also available from:
> > > git://github.com/arm000/u-boot.git
> > > branch: tegra-spl-v7
> > >
> >
> > Applied to u-boot-tegra/next AOK, tested on my Seaboard AOK, so:
> > Tested-by: Tom Warren 
> >
> > Note that I was confused by the final binary name (u-boot-dtb-tegra.bin),
> since I'm used to flashing u-boot-dtb.bin.
> >
> > We need to come to a consensus about the final binary name for Tegra U-
> Boot (I'd thought we had, and that it would be u-boot-dtb.bin, since that's
> what most devs are used to looking for in Tegra builds).
> >
> 
> Yeah, I'd like some stability there too.  The -dtb rule is not tegra
> specific, which is why I didn't want to modify or remove it.  I think we're
> the only one that uses it though, so maybe it's not so bad.
> 
> > Also, one nit: I see the 2 sign-on strings (U-Boot SPL 2012.04.xxx, and
> then U-Boot 2012.04.xxx), separated by 2 lines. I think it'd look better if
> you had them one right after the other, i.e. eliminate the extra linefeeds.
> >
> 
> The extra lines come from display_banner() which is ARM generic from the
> main u-boot.  I assume they are there to separate the banner from any junk
> that was on your screen before you rebooted, so it would make sense to move
> them to the SPL banner instead if you have SPL enabled.
> I'll make a separate patch for that in a week after I get back from
> vacation.
> 
> -Allen

Cool, thanks. Until then:

Tegra2 SPL patches have been applied to u-boot-tegra/next & pushed to Denx. I'm 
going to hold off putting it into tegra/master and generating a pull request 
for awhile to allow all Tegra devs to test it, comment, etc., since it's a 
major change to the Tegra build.

If possible, please post any new Tegra changes against tegra/next (i.e. using 
Allen's SPL file locations).

Thanks,

Tom
--
nvpublic
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-18 Thread Allen Martin
On Tue, Jul 17, 2012 at 12:32:53PM -0700, Tom Warren wrote:
> Allen,
> 
> > -Original Message-
> > From: Allen Martin [mailto:amar...@nvidia.com]
> > Sent: Monday, July 16, 2012 4:02 PM
> > To: Tom Warren; swar...@wwwdotorg.org; s...@chromium.org;
> > thierry.red...@avionic-design.de
> > Cc: u-boot@lists.denx.de; Allen Martin
> > Subject: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
> >
> > This patch series fixes a long standing problem with the tegra20 u-boot
> > build.  Tegra20 contains an ARM7TDMI boot processor and a Cortex A9 main
> > processor.  Prior to this patch series this was accomplished by #ifdefing
> > out any armv7 code from the early boot sequence and creating a single binary
> > that runs on both both the ARM7TDMI and A9.  This was very fragile as
> > changes to compiler options or any additions or rearranging of the early
> > boot code could add additional armv7 specific code causing it to fail on the
> > ARM7TDMI.
> >
> > This patch series pulls all the armv4t code out into a separate SPL that
> > does nothing more than initialize the A9 and transfer control to it.  The
> > resultint SPL and armv7 u-boot are concatenated together into a single
> > image.
> >
> > This patch series is also available from:
> > git://github.com/arm000/u-boot.git
> > branch: tegra-spl-v7
> >
> 
> Applied to u-boot-tegra/next AOK, tested on my Seaboard AOK, so:
> Tested-by: Tom Warren 
> 
> Note that I was confused by the final binary name (u-boot-dtb-tegra.bin), 
> since I'm used to flashing u-boot-dtb.bin.
> 
> We need to come to a consensus about the final binary name for Tegra U-Boot 
> (I'd thought we had, and that it would be u-boot-dtb.bin, since that's what 
> most devs are used to looking for in Tegra builds).
> 

Yeah, I'd like some stability there too.  The -dtb rule is not tegra
specific, which is why I didn't want to modify or remove it.  I think
we're the only one that uses it though, so maybe it's not so bad.

> Also, one nit: I see the 2 sign-on strings (U-Boot SPL 2012.04.xxx, and then 
> U-Boot 2012.04.xxx), separated by 2 lines. I think it'd look better if you 
> had them one right after the other, i.e. eliminate the extra linefeeds.
> 

The extra lines come from display_banner() which is ARM generic from
the main u-boot.  I assume they are there to separate the banner from
any junk that was on your screen before you rebooted, so it would make
sense to move them to the SPL banner instead if you have SPL enabled.
I'll make a separate patch for that in a week after I get back from
vacation. 

-Allen
-- 
nvpublic
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-17 Thread Tom Warren
Allen,

> -Original Message-
> From: Allen Martin [mailto:amar...@nvidia.com]
> Sent: Monday, July 16, 2012 4:02 PM
> To: Tom Warren; swar...@wwwdotorg.org; s...@chromium.org;
> thierry.red...@avionic-design.de
> Cc: u-boot@lists.denx.de; Allen Martin
> Subject: [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
>
> This patch series fixes a long standing problem with the tegra20 u-boot
> build.  Tegra20 contains an ARM7TDMI boot processor and a Cortex A9 main
> processor.  Prior to this patch series this was accomplished by #ifdefing
> out any armv7 code from the early boot sequence and creating a single binary
> that runs on both both the ARM7TDMI and A9.  This was very fragile as
> changes to compiler options or any additions or rearranging of the early
> boot code could add additional armv7 specific code causing it to fail on the
> ARM7TDMI.
>
> This patch series pulls all the armv4t code out into a separate SPL that
> does nothing more than initialize the A9 and transfer control to it.  The
> resultint SPL and armv7 u-boot are concatenated together into a single
> image.
>
> This patch series is also available from:
> git://github.com/arm000/u-boot.git
> branch: tegra-spl-v7
>

Applied to u-boot-tegra/next AOK, tested on my Seaboard AOK, so:
Tested-by: Tom Warren 

Note that I was confused by the final binary name (u-boot-dtb-tegra.bin), since 
I'm used to flashing u-boot-dtb.bin.

We need to come to a consensus about the final binary name for Tegra U-Boot 
(I'd thought we had, and that it would be u-boot-dtb.bin, since that's what 
most devs are used to looking for in Tegra builds).

Also, one nit: I see the 2 sign-on strings (U-Boot SPL 2012.04.xxx, and then 
U-Boot 2012.04.xxx), separated by 2 lines. I think it'd look better if you had 
them one right after the other, i.e. eliminate the extra linefeeds.

Tom
> Changes:
> v7:
>  - rebased to u-boot-tegra/next
>  - added Acks and Tested-Bys
> v6:
>  - changed combined SPL/u-boot target names to u-boot-dtb-tegra.bin  and u-
> boot-nodtb-tegra.bin
>  - fixed regression introduced in v5 that caused git bisect to fail to
> build at one of the patches
>  - fix USE_PRIVATE_LIBGCC for SPL build and enable it
>  - fix libtegra20-common.so link so it works if building with out of  tree
> OBJDIR
> v5:
>  - added missing mkdir rules in all tegra20 board Makefiles that  include
> common code and changed them to be unconditional
>  - make new u-boot-dtb-t2.bin rule default for tegra20 and copy  resulting
> binary over u-boot.bin
> v4:
>  - rebased to u-boot-tegra/next
>  - reordered entire patch series to preserve git bisect, verified  build and
> boot at each patch
>  - merged patches that add SPL config defines and removes duplicate  code
> from u-boot init, these have to go in atomically to preserve  building at
> each patch
>  - fixed compiler warnings introduced
>  - fixed blank line at end of file on cpu.c
>  - renamed u-boot.t2 to u-boot-t2.bin
>  - cleaned up config.mk generation in mkconfig to make it more  readable
>  - added some text to clarify using arm720t code for arm7tdmi
>  - rearranged SPL memory map to make resulting SPL + u-boot image much
> smaller
>  - removed separate PAD_TO define in favor of just using
> CONFIG_SYS_TEXT_BASE
>  - moved warmboot_save_sdram_params() from dram_init() to board_init()
> v3:
>  - git bisect still does not work across this series, I'm saving that  for
> the next revision, but I had enough changes that I wanted to get  this out
> for review
>  - expanded the tegra2 -> tegra20 rename to include  functions/variables/
> defines
>  - rebased to u-boot-tegra/next
>  - removed some extra -march=armv4t flags, kept armv4t flags on
> warmboot_avp since it's special
>  - removed bashisms from mkconfig
>  - renamed CONFIG_MACH_TEGRA_GENERIC to CONFIG_TEGRA
>  - moved SPL overrides to tegra2-common-post.h
>  - changed SPL base address to 0x108000, u-boot goes to 0x208000
>  - moved warboot_save_sdram_params fix to separate patch
>  - remove USE_PRIVATE_LIBGCC from non SPL build
>  - expanded SPL support to all tegra20 boards, not just seaboard
> v2:
>  - renamed tegra2 to tegra20 to match kernel and devicetree naming  policy
>  - pulled all SPL related config overrides to a separate file to clean  up
> ifdefs from seabard.h
>  - rebased to TOT u-boot/master and fixed a bug related to init  sequence
> changes between this patch series and new EMC code
>  - made u-boot.t2 target work even if CONFIG_OF is disabled
>  - added back USE_PRIVATE_LIBGCC
>
> Allen Martin (15):
>   tegra20: rename tegra2 -> tegra20
>   tegra20: move tegra20 SoC code to arch/arm/cpu/tegra20-common
>   tegra20: rename CONFIG_MACH_TEGRA_GENERIC
>   tegra20: tec: add tegra20-common-post.h
>   tegra20: make board mkdir commands unconditional
>   mkconfig: add support for SPL CPU
>   ARM: Fix arm720t SPL build
>   tegra20: remove timer_init from SPL build
>   ARM: add tegra20 support to arm720t
>   tegra20: add u-boot-*-tegra.bin targets
>  

[U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL

2012-07-16 Thread Allen Martin
This patch series fixes a long standing problem with the tegra20
u-boot build.  Tegra20 contains an ARM7TDMI boot processor and a
Cortex A9 main processor.  Prior to this patch series this was
accomplished by #ifdefing out any armv7 code from the early boot
sequence and creating a single binary that runs on both both the
ARM7TDMI and A9.  This was very fragile as changes to compiler options
or any additions or rearranging of the early boot code could add
additional armv7 specific code causing it to fail on the ARM7TDMI.

This patch series pulls all the armv4t code out into a separate SPL
that does nothing more than initialize the A9 and transfer control to
it.  The resultint SPL and armv7 u-boot are concatenated together into
a single image.

This patch series is also available from:
git://github.com/arm000/u-boot.git
branch: tegra-spl-v7

Changes:
v7:
 - rebased to u-boot-tegra/next
 - added Acks and Tested-Bys
v6:
 - changed combined SPL/u-boot target names to u-boot-dtb-tegra.bin
 and u-boot-nodtb-tegra.bin
 - fixed regression introduced in v5 that caused git bisect to fail to
 build at one of the patches
 - fix USE_PRIVATE_LIBGCC for SPL build and enable it
 - fix libtegra20-common.so link so it works if building with out of
 tree OBJDIR
v5:
 - added missing mkdir rules in all tegra20 board Makefiles that
 include common code and changed them to be unconditional
 - make new u-boot-dtb-t2.bin rule default for tegra20 and copy
 resulting binary over u-boot.bin
v4:
 - rebased to u-boot-tegra/next
 - reordered entire patch series to preserve git bisect, verified
 build and boot at each patch
 - merged patches that add SPL config defines and removes duplicate
 code from u-boot init, these have to go in atomically to preserve
 building at each patch
 - fixed compiler warnings introduced
 - fixed blank line at end of file on cpu.c
 - renamed u-boot.t2 to u-boot-t2.bin
 - cleaned up config.mk generation in mkconfig to make it more
 readable
 - added some text to clarify using arm720t code for arm7tdmi
 - rearranged SPL memory map to make resulting SPL + u-boot image much
 smaller 
 - removed separate PAD_TO define in favor of just using
 CONFIG_SYS_TEXT_BASE 
 - moved warmboot_save_sdram_params() from dram_init() to board_init()
v3:
 - git bisect still does not work across this series, I'm saving that
 for the next revision, but I had enough changes that I wanted to get
 this out for review
 - expanded the tegra2 -> tegra20 rename to include
 functions/variables/
 defines
 - rebased to u-boot-tegra/next
 - removed some extra -march=armv4t flags, kept armv4t flags on
 warmboot_avp since it's special
 - removed bashisms from mkconfig
 - renamed CONFIG_MACH_TEGRA_GENERIC to CONFIG_TEGRA
 - moved SPL overrides to tegra2-common-post.h
 - changed SPL base address to 0x108000, u-boot goes to 0x208000
 - moved warboot_save_sdram_params fix to separate patch
 - remove USE_PRIVATE_LIBGCC from non SPL build
 - expanded SPL support to all tegra20 boards, not just seaboard
v2:
 - renamed tegra2 to tegra20 to match kernel and devicetree naming
 policy
 - pulled all SPL related config overrides to a separate file to clean
 up ifdefs from seabard.h
 - rebased to TOT u-boot/master and fixed a bug related to init
 sequence changes between this patch series and new EMC code
 - made u-boot.t2 target work even if CONFIG_OF is disabled
 - added back USE_PRIVATE_LIBGCC

Allen Martin (15):
  tegra20: rename tegra2 -> tegra20
  tegra20: move tegra20 SoC code to arch/arm/cpu/tegra20-common
  tegra20: rename CONFIG_MACH_TEGRA_GENERIC
  tegra20: tec: add tegra20-common-post.h
  tegra20: make board mkdir commands unconditional
  mkconfig: add support for SPL CPU
  ARM: Fix arm720t SPL build
  tegra20: remove timer_init from SPL build
  ARM: add tegra20 support to arm720t
  tegra20: add u-boot-*-tegra.bin targets
  tegra20: move SDRAM param save to later in boot
  tegra20: enable SPL for tegra20 boards
  arm: enable libgcc build for SPL
  spl: fix SPL build of private libgcc
  tegra20: Remove armv4t build flags

 MAINTAINERS|   18 +--
 Makefile   |   26 
 arch/arm/cpu/arm720t/cpu.c |2 +
 arch/arm/cpu/arm720t/interrupts.c  |5 +
 arch/arm/cpu/arm720t/start.S   |   19 ++-
 .../arm/cpu/arm720t/tegra20}/Makefile  |   27 ++--
 .../arch-tegra2 => cpu/arm720t/tegra20}/board.h|   11 +-
 .../{armv7/tegra2 => arm720t/tegra20}/config.mk|   12 --
 .../{armv7/tegra2/ap20.c => arm720t/tegra20/cpu.c} |  148 ++--
 .../ap20.h => cpu/arm720t/tegra20/cpu.h}   |   15 +-
 arch/arm/cpu/arm720t/tegra20/spl.c |  132 +
 arch/arm/cpu/armv7/start.S |2 -
 arch/arm/cpu/armv7/{tegra2 => tegra20}/Makefile|   15 +-
 .../cpu/armv7/{tegra2 => tegra20}/cmd_enterrcm.c   |4 +-
 arch/arm/cpu/armv7/{tegra2 => tegra20}/c