From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Martin Date: Mon, 11 Jun 2012 12:01:32 -0700 Subject: [U-Boot] [PATCH v3 07/14] ARM: add tegra20 support to arm720t In-Reply-To: References: <1339190167-20320-1-git-send-email-amartin@nvidia.com> <1339190167-20320-8-git-send-email-amartin@nvidia.com> Message-ID: <20120611190132.GD6633@nvidia.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sat, Jun 09, 2012 at 11:32:08AM -0700, Simon Glass wrote: > Hi Allen, > > On Fri, Jun 8, 2012 at 2:16 PM, Allen Martin > wrote: > Add support for tegra20 arm7 boot processor. This processor is used > to power on the Cortex A9 and transfer control to it. > > Signed-off-by: Allen Martin > > --- > arch/arm/cpu/arm720t/cpu.c | 2 + > arch/arm/cpu/arm720t/interrupts.c | 4 + > arch/arm/cpu/arm720t/start.S | 6 +- > .../{tegra20-common => arm720t/tegra20}/Makefile | 22 +- > 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/tegra20-common/Makefile | 2 +- > arch/arm/include/asm/arch-tegra20/hardware.h | 29 +++ > > I think this (empty) file is required by arm720t/cpu.c, right? Yes. > > You are using arm720t here, but your chip is actually an ARM7TDMI. I > suppose that doesn't matter since there is no cache and they share > the same core. But perhaps a comment would be useful explaining this > difference and how it was convenient to use the same code. Good point, will do. -Allen -- nvpublic