split tegra20 arm7 code into separate SPL

boards.cfg

Quoted 4 lines · Allen Martin

This is make naming consistent with the kernel and devicetree and in preparation of pulling out the common tegra20 code.

boards.cfg +0 -0

This misses a couple of recently added boards - whistler and trimslice.

Quoted 5 lines · Allen Martin

In preparation for splitting out the armv4t code from tegra20, move the tegra20 SoC code to arch/arm/cpu/tegra20-common. This code will be compiled armv4t for the arm7tdmi and armv7 for the cortex A9.

arch/arm/cpu/tegra20-common/Makefile renamed from arch/arm/cpu/armv7/tegra20/Makefile +0 -0

arch/arm/cpu/tegra20-common/Makefile line 31

Quoted 7 lines

# The AVP is ARMv4T architecture so we must use special compiler

 # flags for any startup files it might use.

arch/arm/cpu/tegra20-common/Makefile renamed from arch/arm/cpu/armv7/tegra20/Makefile +2 -3

-CFLAGS_arch/arm/cpu/armv7/tegra2/ap20.o += -march=armv4t
-CFLAGS_arch/arm/cpu/armv7/tegra2/clock.o += -march=armv4t
-CFLAGS_arch/arm/cpu/armv7/tegra2/warmboot_avp.o += -march=armv4t
+CFLAGS_arch/arm/cpu/tegra2-common/ap20.o += -march=armv4t
+CFLAGS_arch/arm/cpu/tegra2-common/clock.o += -march=armv4t

Don't you still need the CFLAGS override for warmboot_avp.o here too? I assume that a later patch will just remove it, but it'd be nice if each patch along the way built and booted OK to, so that "git bisect" works through this series.

arch/arm/cpu/tegra20-common/Makefile line 42

Quoted 5 lines

arch/arm/cpu/tegra20-common/Makefile renamed from arch/arm/cpu/armv7/tegra20/Makefile +2 -3

-COBJS	:= $(COBJS-y)
-SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS	:= $(addprefix $(obj),$(COBJS) $(SOBJS))
+SRCS	:= $(SOBJS:.o=.S) $(COBJS-y:.o=.c)
+OBJS	:= $(addprefix $(obj),$(SOBJS) $(COBJS-y))

At least in this patch, I don't think SOBJS is set. Does using it here cause build issues?

spl/Makefile ifneq ($(CONFIG_AM33XX)$(CONFIG_OMAP34XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),) · line 67

Quoted 5 lines

spl/Makefile +3 -0

 
+ifneq ($(CONFIG_TEGRA2),)
+LIBS-y += arch/$(ARCH)/cpu/$(SOC)-common/lib$(SOC)-common.o
+endif

In another file in this patch, that first line was the non-inverted test:

+ifeq ($(SOC),tegra20)

That seems simpler. Can we use the same here?

Add support for specifying a differnt CPU for main u-boot and SPL

u-boot builds. This is done by adding an optional SPL CPU after the main CPU in boards.cfg as follows:

normal_cpu:spl_cpu

This this case CPU will be set to "normal_cpu" during the main u-boot build and "spl_cpu" during the SPL build.

Quoted 1 line · Allen Martin

Add support for specifying a differnt CPU for main u-boot and SPL

typo ^^

Quoted 9 lines

u-boot builds. This is done by adding an optional SPL CPU after the main CPU in boards.cfg as follows:

normal_cpu:spl_cpu

This this case CPU will be set to "normal_cpu" during the main u-boot build and "spl_cpu" during the SPL build.

mkconfig CONFIG_NAME="${1%_config}" · line 67

Quoted 6 lines

mkconfig +5 -0

 
+tmp="${cpu#*:}"
+if [ "$tmp" != "$cpu" ] ; then
+	spl_cpu=$tmp
+	cpu="${cpu%:*}"
+fi

Neat - I wasn't aware of those variable expansion techniques.

However, this script runs under /bin/sh - are they standard sh constructs, or are they bash-isms? I wonder if shelling out to awk or cut here would be more portable?

+void init_pllx(void)

Quoted 4 lines · Allen Martin

Add support for tegra20 arm7 boot processor. This processor is used to power on the Cortex A9 and transfer control to it.

arch/arm/cpu/arm720t/cpu.c +0 -0

arch/arm/cpu/arm720t/cpu.c int cleanup_before_linux (void) · line 58

Quoted 7 lines

#elif defined(CONFIG_INTEGRATOR) && defined(CONFIG_ARCH_INTEGRATOR)

arch/arm/cpu/arm720t/cpu.c +2 -0

 	/* No cleanup before linux for IntegratorAP/CM720T as yet */
+#elif defined (CONFIG_MACH_TEGRA_GENERIC)
+	/* No cleanup before linux for tegra as yet */
 #else
 #error No cleanup_before_linux() defined for this CPU type
 #endif

Wouldn't CONFIG_TEGRA2 be a better define to use? Same comment throughout.

arch/arm/cpu/arm720t/tegra20/cpu.c line 39

Quoted 8 lines

arch/arm/cpu/arm720t/tegra20/cpu.c +6 -0

 
+/* Returns 1 if the current CPU executing is a Cortex-A9, else 0 */
+int ap20_cpu_is_cortexa9(void)
+{
+	u32 id = readb(NV_PA_PG_UP_BASE + PG_UP_TAG_0);
+	return id == (PG_UP_TAG_0_PID_CPU & 0xff);
+}

Doesn't the SPL always run on the AVP, and the regular U-Boot build always run on the main CPUs - I'm not sure why this runtime API would be needed.

Quoted 1 line

+void init_pllx(void)

I assume a lot of the code in this file is copied from the armv7/tegra20/*.c files. Shouldn't this patch remove the ARMv7 versions since presumably it's no longer needed - doesn't the non-SPL U-Boot rely on the SPL U-Boot always having run first in order to start the main U-Boot executing?

If not, then shouldn't this code be in tegra20-common so it can be shared?

arch/arm/cpu/arm720t/tegra20/cpu.h

Quoted 1 line

arch/arm/cpu/arm720t/tegra20/cpu.h +0 -0

Similarly, this file seems like it'd be in tegra20-common/include or somewhere like that.

Quoted 6 lines · Stephen Warren

Quoted 4 lines · Allen Martin

Add support for tegra20 arm7 boot processor. This processor is used to power on the Cortex A9 and transfer control to it.

arch/arm/cpu/arm720t/cpu.c +0 -0

Quoted 9 lines

arch/arm/cpu/arm720t/cpu.c int cleanup_before_linux (void) · line 58

Quoted 7 lines

#elif defined(CONFIG_INTEGRATOR) && defined(CONFIG_ARCH_INTEGRATOR)

arch/arm/cpu/arm720t/cpu.c +2 -0

 	/* No cleanup before linux for IntegratorAP/CM720T as yet */
+#elif defined (CONFIG_MACH_TEGRA_GENERIC)
+	/* No cleanup before linux for tegra as yet */
 #else
 #error No cleanup_before_linux() defined for this CPU type
 #endif

Wouldn't CONFIG_TEGRA2 be a better define to use? Same comment throughout.

The SPL *should* work on tegra30 with only minor changes, so just trying to look forward.

Oh, I see. MACH_TEGRA_GENERIC means "some Tegra SoC", and CONFIG_TEGRA* indicate which one. Coming from my kernel background, MACH_* is a define for a particular board, and we do have a GENERIC Tegra board, so I got the wrong end of the stick.

I wonder if it's worth renaming CONFIG_MACH_TEGRA_GENERIC to CONFIG_TEGRA. There don't appear to be any users of it currently.

Quoted 7 lines · Stephen Warren

Oh, I see. MACH_TEGRA_GENERIC means "some Tegra SoC", and CONFIG_TEGRA* indicate which one. Coming from my kernel background, MACH_* is a define for a particular board, and we do have a GENERIC Tegra board, so I got the wrong end of the stick.

I wonder if it's worth renaming CONFIG_MACH_TEGRA_GENERIC to CONFIG_TEGRA. There don't appear to be any users of it currently.

Fine by me, maybe CONFIG_TEGRA2 should go to CONFIG_TEGRA20 also to keep with the new naming?

include/configs/tegra2-common.h +4 -0

 
+/* Defines for SPL */
+#define CONFIG_SPL
+#define CONFIG_SPL_NAND_SIMPLE
+#define CONFIG_SPL_TEXT_BASE		0x00008000

+#define CONFIG_SPL_GPIO_SUPPORT

Quoted 4 lines · Allen Martin

Add SPL options to tegra20 config files and enable SPL build for seaboard in boards.cfg

include/configs/seaboard.h +0 -0

include/configs/seaboard.h line 115

Quoted 4 lines

include/configs/seaboard.h +4 -0

+/* include overrides for SPL build */
+#ifdef CONFIG_SPL_BUILD
+#include "tegra2-spl.h"
+#endif

Perhaps we should add that to include/configs/tegra2-common-post.h, which I think all Tegra config headers now include at the end.

Quoted 15 lines

include/configs/tegra2-common.h +12 -0

 
+/* Defines for SPL */
+#define CONFIG_SPL
+#define CONFIG_SPL_NAND_SIMPLE
+#define CONFIG_SPL_TEXT_BASE		0x00008000
+#define CONFIG_SPL_MAX_SIZE		0x00078000
+#define CONFIG_SYS_SPL_MALLOC_START	0x00080000
+#define CONFIG_SYS_SPL_MALLOC_SIZE	0x00010000
+#define CONFIG_SPL_STACK		0x0009fffc
+
+#define CONFIG_SPL_LIBCOMMON_SUPPORT
+#define CONFIG_SPL_LIBGENERIC_SUPPORT
+#define CONFIG_SPL_SERIAL_SUPPORT

Quoted 1 line

+#define CONFIG_SPL_POWER_SUPPORT

I'm not sure what that's for?

Quoted 1 line

+#define CONFIG_SPL_GPIO_SUPPORT

I believe only Seaboard needs that, not all boards. It'd be nice if we could just ignore the crufy SPI/GPIO switch on Seaboard too and punt this. Does anyone actually boot Seaboard using SPI? I use NAND and Tom uses eMMC...

include/configs/tegra2-spl.h line 11

Quoted 4 lines

include/configs/tegra2-spl.h +2 -0

 
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.

Being a new file, this should be GPLv2 not GPLv2+.

include/configs/tegra2-spl.h line 30

Quoted 5 lines

include/configs/tegra2-spl.h +4 -0

+/* remove devicetree support */
+#ifdef CONFIG_OF_CONTROL
+#undef CONFIG_OF_CONTROL
+#endif

I don't /think/ #undef complains if the define isn't defined?

Perhaps this (and the others) are better approached by not defining all these if building SPL though - what if a particular board's SPL actually wanted any of these?

On 06/05/2012 03:20 PM, Allen Martin wrote:

include/configs/tegra2-spl.h line 11

Quoted 4 lines

include/configs/tegra2-spl.h +2 -0

 
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.

Being a new file, this should be GPLv2 not GPLv2+.

Quoted 1 line · Stephen Warren

On 06/05/2012 03:20 PM, Allen Martin wrote:

<snip>

Quoted 5 lines

Quoted 4 lines

include/configs/tegra2-spl.h +2 -0

 
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.

Quoted 1 line

Being a new file, this should be GPLv2 not GPLv2+.

u-boot uses GPLv2+.

http://www.denx.de/wiki/view/U-Boot/Patches#Attributing_Code_Copyrights_Sign
http://www.mail-archive.com/u-boot at lists.denx.de/msg26569.html

U-Boot is Free Software. It is copyrighted by Wolfgang Denk and
many others who contributed code (see the actual source code for details). You can redistribute U-Boot and/or modify it under the terms of version 2 of the GNU General Public License as published by the Free Software Foundation. Most of it can also be distributed, at your option, under any later version of the GNU General Public License -- see individual files for exceptions.

Quoted 10 lines · Sughosh Ganu

Quoted 1 line · Stephen Warren

On 06/05/2012 03:20 PM, Allen Martin wrote:

<snip>

Quoted 5 lines

Quoted 4 lines

include/configs/tegra2-spl.h +2 -0

 
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.

Quoted 6 lines

Quoted 1 line

Being a new file, this should be GPLv2 not GPLv2+.

u-boot uses GPLv2+.

http://www.denx.de/wiki/view/U-Boot/Patches#Attributing_Code_Copyrights_Sign
http://www.mail-archive.com/u-boot at lists.denx.de/msg26569.html

Sughosh,

I guess I'm a little confused here.

COPYING in the U-Boot source tree states the following:

Quoted 7 lines

U-Boot is Free Software. It is copyrighted by Wolfgang Denk and
many others who contributed code (see the actual source code for details). You can redistribute U-Boot and/or modify it under the terms of version 2 of the GNU General Public License as published by the Free Software Foundation. Most of it can also be distributed, at your option, under any later version of the GNU General Public License -- see individual files for exceptions.

... which explicitly allows for either GPLv2 or GPLv2+ code. There is certainly GPLv2-only code already in U-Boot. Why do you say that GPLv2-only code isn't allowed?

Quoted 32 lines · Stephen Warren

Quoted 10 lines · Sughosh Ganu

Quoted 1 line · Stephen Warren

On 06/05/2012 03:20 PM, Allen Martin wrote:

<snip>

Quoted 5 lines

Quoted 4 lines

include/configs/tegra2-spl.h +2 -0

 
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.

Quoted 6 lines

Quoted 1 line

Being a new file, this should be GPLv2 not GPLv2+.

u-boot uses GPLv2+.

http://www.denx.de/wiki/view/U-Boot/Patches#Attributing_Code_Copyrights_Sign
http://www.mail-archive.com/u-boot at lists.denx.de/msg26569.html

Sughosh,

I guess I'm a little confused here.

COPYING in the U-Boot source tree states the following:

Quoted 7 lines

U-Boot is Free Software. It is copyrighted by Wolfgang Denk and
many others who contributed code (see the actual source code for details). You can redistribute U-Boot and/or modify it under the terms of version 2 of the GNU General Public License as published by the Free Software Foundation. Most of it can also be distributed, at your option, under any later version of the GNU General Public License -- see individual files for exceptions.

Quoted 3 lines

... which explicitly allows for either GPLv2 or GPLv2+ code. There is certainly GPLv2-only code already in U-Boot. Why do you say that GPLv2-only code isn't allowed?

I guess that could be because some of the existing files in u-boot are released under GPLv2 only license -- hence a separate mention that most of the files can be distributed under any later version too . Both Wolfgang and Detlev have stated many a times on the mailing list earlier that all new files need to be released under GPLv2+, and not GPLv2 only.

board/nvidia/seaboard/config.mk

Quoted 6 lines · Allen Martin

Add target for tegra20 u-boot image. This is a concatenation of tegra spl and normal u-boot binaries.

board/nvidia/seaboard/config.mk +1 -0

 
+PAD_TO=0x00108000

Oh crap, does this mean that we have to start flashing the combined u-boot.t2 at 0x8000 (IIRC the start address of the SPL in an earlier patch) instead of 0x108000? That would cause all kinds of problems. If that is the case, can we move the SPL to the existing 0x108000, and bump the regular U-Boot up a bit to make space? If that isn't the case, could you please explain exactly how the SPL memory layout etc. works for me? Thanks.

-----Original Message-----
From: Allen Martin [mailto:amartin at nvidia.com]
Sent: Wednesday, June 06, 2012 12:37 PM
To: Stephen Warren
Cc: Tom Warren; sjg at chromium.org; u-boot at lists.denx.de
Subject: Re: [PATCH v2 08/10] tegra20: add u-boot.t2 target

Quoted 7 lines · Stephen Warren

Quoted 5 lines · Allen Martin

Add target for tegra20 u-boot image. This is a concatenation of tegra spl and normal u-boot binaries.

board/nvidia/seaboard/config.mk +0 -0

Quoted 8 lines

Quoted 1 line

+PAD_TO=0x00108000

Oh crap, does this mean that we have to start flashing the combined u-boot.t2 at 0x8000 (IIRC the start address of the SPL in an earlier patch) instead of 0x108000? That would cause all kinds of problems. If that is the case, can we move the SPL to the existing 0x108000, and bump the regular U-Boot up a bit to make space? If that isn't the case, could you please explain exactly how the SPL memory layout etc.

works for me?

Quoted 1 line

Thanks.

The addresses are arbitrary, I picked 0x8000 for the SPL to preserve the existing 0x108000 for the normal u-boot but it doesn't have to be that way. The only requirement is that the SPL needs to know the address of the normal u-boot at compile time because it doesn't have any smarts to look for it.

I'll move the SPL to 0x108000 and the normal u-boot to 0x208000 if that sounds more acceptable.

-Allen

Allen,

Quoted 31 lines

-----Original Message-----
From: Allen Martin [mailto:amartin at nvidia.com]
Sent: Wednesday, June 06, 2012 12:37 PM
To: Stephen Warren
Cc: Tom Warren; sjg at chromium.org; u-boot at lists.denx.de
Subject: Re: [PATCH v2 08/10] tegra20: add u-boot.t2 target

Quoted 7 lines · Stephen Warren

Quoted 5 lines · Allen Martin

Add target for tegra20 u-boot image. This is a concatenation of tegra spl and normal u-boot binaries.

board/nvidia/seaboard/config.mk +0 -0

Quoted 8 lines

Quoted 1 line

+PAD_TO=0x00108000

Oh crap, does this mean that we have to start flashing the combined u-boot.t2 at 0x8000 (IIRC the start address of the SPL in an earlier patch) instead of 0x108000? That would cause all kinds of problems. If that is the case, can we move the SPL to the existing 0x108000, and bump the regular U-Boot up a bit to make space? If that isn't the case, could you please explain exactly how the SPL memory layout etc.

works for me?

Quoted 1 line

Thanks.

The addresses are arbitrary, I picked 0x8000 for the SPL to preserve the existing 0x108000 for the normal u-boot but it doesn't have to be that way. The only requirement is that the SPL needs to know the address of the normal u-boot at compile time because it doesn't have any smarts to look for it.

Quoted 2 lines

I'll move the SPL to 0x108000 and the normal u-boot to 0x208000 if that sounds more acceptable.

The current flash tools everyone is using expect to flash U-Boot to 0x0108000 (on T20). This works with the non-SPL (current) upstream U-Boot for Tegra, and should continue to work for any new SPL Tegra U-Boot, so that people can use the same flash tools for either build.

Note that the above statement assumes we'll still be able to build either a 'normal' non-SPL build of Tegra U-Boot and a SPL build with your patchset, at least in the interim while we're testing/reviewing these major changes.

Quoted 6 lines · Tom Warren

Quoted 3 lines

I'll move the SPL to 0x108000 and the normal u-boot to 0x208000 if that sounds more acceptable.

The current flash tools everyone is using expect to flash U-Boot to 0x0108000 (on T20). This works with the non-SPL (current) upstream U-Boot for Tegra, and should continue to work for any new SPL Tegra U-Boot, so that people can use the same flash tools for either build.

Note that the above statement assumes we'll still be able to build either a 'normal' non-SPL build of Tegra U-Boot and a SPL build with your patchset, at least in the interim while we're testing/reviewing these major changes.

I assume that once a board is converted to SPL, you always use a matched/concatenated pair of SPL+non-SPL, i.e. never SPL on its own or non-SPL on its own?

But that does bring up one point - boards which haven't yet converted to SPL need to still build/link for the original 00108000 address, so the addresses need to depend on config data.

Either that, or just convert all boards at once?

Quoted 8 lines · Stephen Warren

Quoted 6 lines · Tom Warren

Quoted 3 lines

I'll move the SPL to 0x108000 and the normal u-boot to 0x208000 if that sounds more acceptable.

The current flash tools everyone is using expect to flash U-Boot to 0x0108000 (on T20). This works with the non-SPL (current) upstream U-Boot for Tegra, and should continue to work for any new SPL Tegra U-Boot, so that people can use the same flash tools for either build.

Note that the above statement assumes we'll still be able to build either a 'normal' non-SPL build of Tegra U-Boot and a SPL build with your patchset, at least in the interim while we're testing/reviewing these major changes.

Quoted 3 lines

I assume that once a board is converted to SPL, you always use a matched/concatenated pair of SPL+non-SPL, i.e. never SPL on its own or non-SPL on its own?

Yes, that's the intention. Although I do have plans down the road for adding memory initialization to the SPL so I can use it for boot/flashing when in recovery mode and there's no BCT. In the normal case they have to be used together.

Quoted 4 lines · Allen Martin

Yes, that's the intention. Although I do have plans down the road for adding memory initialization to the SPL so I can use it for boot/flashing when in recovery mode and there's no BCT. In the normal case they have to be used together.

How would that work; in the regular boot process, SPL would be linked to e.g. 00108000 since the boot ROM will put it into SDRAM. However, in recovery mode, code will be downloaded to the IRAM, so would have to be linked to a different address, and hence the same SPL binary couldn't be used. At that point, aren't we looking at:

* Regular boot SPL, which just sets up a few PLLs and jump-starts the A9s.
* Regular boot full-blown U-Boot to run on the A9s.
* Some third build t run from IRAM.

Or am I missing something?

arch/arm/cpu/tegra20-common/ap20.c void init_pmc_scratch(void) · line 94

Quoted 12 lines · Allen Martin

This code is now included in the tegra20 SPL

arch/arm/cpu/tegra20-common/ap20.c +0 -5

@@ -323,34 +91,10 @@ void init_pmc_scratch(void)
 
 	/* ODMDATA is for kernel use to determine RAM size, LP config, etc. */
 	writel(CONFIG_SYS_BOARD_ODMDATA, &pmc->pmc_scratch20);
-
-#ifdef CONFIG_TEGRA2_LP0
-	/* save Sdram params to PMC 2, 4, and 24 for WB0 */
-	warmboot_save_sdram_params();
-#endif
 }

Is that change intended, or was it a rebase mistake?

Quoted 1 line

arch/arm/cpu/tegra20-common/board.c +0 -0

arch/arm/cpu/tegra20-common/board.c int dram_init(void) · line 75

Quoted 9 lines

arch/arm/cpu/tegra20-common/board.c +5 -0

@@ -69,6 +68,11 @@ int dram_init(void)
 {
 	/* We do not initialise DRAM here. We just query the size */
 	gd->ram_size = query_sdram_size();
+
+#ifdef CONFIG_TEGRA2_LP0
+	/* save Sdram params to PMC 2, 4, and 24 for WB0 */
+	warmboot_save_sdram_params();
+#endif

Hmmm. That's more than just removing the code that's now in the AVP directory. Separate patch? The patch description also doesn't say why this change is necessary.

These flags were necessary when building tegra20 as a single binary that supported ARM7TDMI and Cortex A9. Now that the ARM7TDMI support is split into a separate SPL, this is no longer necessary.

-----Original Message-----
From: Allen Martin [mailto:amartin at nvidia.com]
Sent: Tuesday, June 05, 2012 2:20 PM
To: Tom Warren; swarren at wwwdotorg.org; sjg at chromium.org
Cc: u-boot at lists.denx.de
Subject: [PATCH v2 0/10] 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.

Changes:
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

[PATCH 01/10] tegra20: rename tegra2 -> tegra20 [PATCH 02/10] tegra20: move tegra20 SoC code to [PATCH 03/10] mkconfig: add support for SPL CPU [PATCH 04/10] ARM: Fix arm720t SPL build [PATCH 05/10] tegra20: remove timer_init from SPL build [PATCH 06/10] ARM: add tegra20 support to arm720t [PATCH 07/10] tegra20: add SPL config options for seaboard [PATCH 08/10] tegra20:
add u-boot.t2 target [PATCH 09/10] tegra20: Remove CPU init code from tegra20 u-boot [PATCH 10/10] tegra20: Remove armv4t build flags

 .gitignore                                         |    1 -
 Makefile                                           |   14 -
 arch/arm/cpu/arm720t/cpu.c                         |    2 -
 arch/arm/cpu/arm720t/interrupts.c                  |    4 -
 arch/arm/cpu/arm720t/start.S                       |   19 +-
 arch/arm/cpu/arm720t/tegra20/Makefile              |   48 -
 arch/arm/cpu/arm720t/tegra20/board.h               |   25 -
 arch/arm/cpu/arm720t/tegra20/cpu.c                 |  259 -----
 arch/arm/cpu/arm720t/tegra20/cpu.h                 |   99 --
 arch/arm/cpu/arm720t/tegra20/spl.c                 |  133 ---
 arch/arm/cpu/armv7/start.S                         |    2 +
 arch/arm/cpu/armv7/tegra2/Makefile                 |   59 ++
 arch/arm/cpu/armv7/tegra2/ap20.c                   |  366 +++++++
 arch/arm/cpu/armv7/tegra2/board.c                  |  151 +++
 arch/arm/cpu/armv7/tegra2/clock.c                  | 1087
++++++++++++++++++++
 arch/arm/cpu/armv7/tegra2/config.mk                |   35 +
 arch/arm/cpu/armv7/tegra2/crypto.c                 |  230 +++++
 arch/arm/cpu/armv7/tegra2/crypto.h                 |   36 +
 arch/arm/cpu/armv7/tegra2/emc.c                    |  286 +++++
 arch/arm/cpu/armv7/tegra2/funcmux.c                |  200 ++++
 arch/arm/cpu/armv7/tegra2/lowlevel_init.S          |   42 +
 arch/arm/cpu/armv7/tegra2/pinmux.c                 |  572 ++++++++++
 arch/arm/cpu/armv7/tegra2/pmu.c                    |   70 ++
 arch/arm/cpu/armv7/tegra2/sys_info.c               |   35 +
 arch/arm/cpu/armv7/tegra2/timer.c                  |  111 ++
 arch/arm/cpu/armv7/tegra2/usb.c                    |  460 +++++++++
 arch/arm/cpu/armv7/tegra2/warmboot.c               |  386 +++++++
 arch/arm/cpu/armv7/tegra2/warmboot_avp.c           |  250 +++++
 arch/arm/cpu/armv7/tegra2/warmboot_avp.h           |   81 ++
 arch/arm/cpu/armv7/tegra20/Makefile                |   48 -
 arch/arm/cpu/armv7/tegra20/config.mk               |   28 -
 arch/arm/cpu/armv7/tegra20/usb.c                   |  460 ---------
 arch/arm/cpu/tegra20-common/Makefile               |   59 --
 arch/arm/cpu/tegra20-common/ap20.c                 |  110 --
 arch/arm/cpu/tegra20-common/board.c                |  134 ---
 arch/arm/cpu/tegra20-common/clock.c                | 1087 -----------------
---
 arch/arm/cpu/tegra20-common/crypto.c               |  230 -----
 arch/arm/cpu/tegra20-common/crypto.h               |   36 -
 arch/arm/cpu/tegra20-common/emc.c                  |  286 -----
 arch/arm/cpu/tegra20-common/funcmux.c              |  200 ----
 arch/arm/cpu/tegra20-common/lowlevel_init.S        |   42 -
 arch/arm/cpu/tegra20-common/pinmux.c               |  572 ----------
 arch/arm/cpu/tegra20-common/pmu.c                  |   70 --
 arch/arm/cpu/tegra20-common/sys_info.c             |   35 -
 arch/arm/cpu/tegra20-common/timer.c                |  111 --
 arch/arm/cpu/tegra20-common/warmboot.c             |  386 -------
 arch/arm/cpu/tegra20-common/warmboot_avp.c         |  250 -----
 arch/arm/cpu/tegra20-common/warmboot_avp.h         |   81 --
 arch/arm/include/asm/arch-tegra2/ap20.h            |  109 ++
 arch/arm/include/asm/arch-tegra2/apb_misc.h        |   36 +
 arch/arm/include/asm/arch-tegra2/board.h           |   30 +
 arch/arm/include/asm/arch-tegra2/clk_rst.h         |  147 +++
 arch/arm/include/asm/arch-tegra2/clock.h           |  406 ++++++++
 arch/arm/include/asm/arch-tegra2/emc.h             |  113 ++
 arch/arm/include/asm/arch-tegra2/flow.h            |   36 +
 arch/arm/include/asm/arch-tegra2/funcmux.h         |   69 ++
 arch/arm/include/asm/arch-tegra2/fuse.h            |   39 +
 arch/arm/include/asm/arch-tegra2/gp_padctrl.h      |   73 ++
 arch/arm/include/asm/arch-tegra2/gpio.h            |  289 ++++++
 arch/arm/include/asm/arch-tegra2/mmc.h             |   27 +
 arch/arm/include/asm/arch-tegra2/pinmux.h          |  354 +++++++
 arch/arm/include/asm/arch-tegra2/pmc.h             |  132 +++
 arch/arm/include/asm/arch-tegra2/pmu.h             |   30 +
 arch/arm/include/asm/arch-tegra2/scu.h             |   43 +
 arch/arm/include/asm/arch-tegra2/sdram_param.h     |  148 +++
 arch/arm/include/asm/arch-tegra2/sys_proto.h       |   35 +
 arch/arm/include/asm/arch-tegra2/tegra2.h          |   86 ++
 arch/arm/include/asm/arch-tegra2/tegra2_spi.h      |   76 ++
 arch/arm/include/asm/arch-tegra2/tegra_i2c.h       |  164 +++
 arch/arm/include/asm/arch-tegra2/timer.h           |   30 +
 arch/arm/include/asm/arch-tegra2/uart-spi-switch.h |   46 +
 arch/arm/include/asm/arch-tegra2/uart.h            |   47 +
 arch/arm/include/asm/arch-tegra2/usb.h             |  252 +++++
 arch/arm/include/asm/arch-tegra2/warmboot.h        |  150 +++
 arch/arm/include/asm/arch-tegra20/ap20.h           |  109 --
 arch/arm/include/asm/arch-tegra20/apb_misc.h       |   36 -
 arch/arm/include/asm/arch-tegra20/board.h          |   30 -
 arch/arm/include/asm/arch-tegra20/clk_rst.h        |  147 ---
 arch/arm/include/asm/arch-tegra20/clock.h          |  406 --------
 arch/arm/include/asm/arch-tegra20/emc.h            |  113 --
 arch/arm/include/asm/arch-tegra20/flow.h           |   36 -
 arch/arm/include/asm/arch-tegra20/funcmux.h        |   69 --
 arch/arm/include/asm/arch-tegra20/fuse.h           |   39 -
 arch/arm/include/asm/arch-tegra20/gp_padctrl.h     |   73 --
 arch/arm/include/asm/arch-tegra20/gpio.h           |  289 ------
 arch/arm/include/asm/arch-tegra20/hardware.h       |   29 -
 arch/arm/include/asm/arch-tegra20/mmc.h            |   27 -
 arch/arm/include/asm/arch-tegra20/pinmux.h         |  354 -------
 arch/arm/include/asm/arch-tegra20/pmc.h            |  132 ---
 arch/arm/include/asm/arch-tegra20/pmu.h            |   30 -
 arch/arm/include/asm/arch-tegra20/scu.h            |   43 -
 arch/arm/include/asm/arch-tegra20/sdram_param.h    |  148 ---
 arch/arm/include/asm/arch-tegra20/sys_proto.h      |   35 -
 arch/arm/include/asm/arch-tegra20/tegra2.h         |   86 --
 arch/arm/include/asm/arch-tegra20/tegra2_spi.h     |   76 --
 arch/arm/include/asm/arch-tegra20/tegra_i2c.h      |  164 ---
 arch/arm/include/asm/arch-tegra20/timer.h          |   30 -
 .../arm/include/asm/arch-tegra20/uart-spi-switch.h |   46 -
 arch/arm/include/asm/arch-tegra20/uart.h           |   47 -
 arch/arm/include/asm/arch-tegra20/usb.h            |  252 -----
 arch/arm/include/asm/arch-tegra20/warmboot.h       |  150 ---
 board/nvidia/common/board.c                        |    2 -
 board/nvidia/seaboard/config.mk                    |    1 -

 boards.cfg                                         |   17 +-
 doc/README.SPL                                     |   12 -
 include/configs/seaboard.h                         |    6 -
 include/configs/tegra2-common.h                    |   21 +-
 include/configs/tegra2-spl.h                       |   87 --
 mkconfig                                           |   15 +-
 spl/Makefile                                       |    4 -
 110 files changed, 7438 insertions(+), 7978 deletions(-)

Quoted 1 line

Did you use the -C option for format-patch? I'd expect a lot of these file moves to show up as simple renames.