From: Allen Martin <amartin@nvidia.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL
Date: Wed, 18 Jul 2012 17:02:00 -0700 [thread overview]
Message-ID: <20120719000200.GC20487@nvidia.com> (raw)
In-Reply-To: <5FBF8E85CA34454794F0F7ECBA79798F379D86EB24@HQMAIL04.nvidia.com>
On Tue, Jul 17, 2012 at 12:32:53PM -0700, Tom Warren wrote:
> Allen,
>
> > -----Original Message-----
> > From: Allen Martin [mailto:amartin at nvidia.com]
> > Sent: Monday, July 16, 2012 4:02 PM
> > To: Tom Warren; swarren at wwwdotorg.org; sjg at chromium.org;
> > thierry.reding at avionic-design.de
> > Cc: u-boot at 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 <twarren@nvidia.com>
>
> 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
next prev parent reply other threads:[~2012-07-19 0:02 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-16 23:02 [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 01/15] tegra20: rename tegra2 -> tegra20 Allen Martin
2012-07-19 11:30 ` Simon Glass
2012-07-16 23:02 ` [U-Boot] [PATCH v7 02/15] tegra20: move tegra20 SoC code to arch/arm/cpu/tegra20-common Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 03/15] tegra20: rename CONFIG_MACH_TEGRA_GENERIC Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 04/15] tegra20: tec: add tegra20-common-post.h Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 05/15] tegra20: make board mkdir commands unconditional Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 06/15] mkconfig: add support for SPL CPU Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 07/15] ARM: Fix arm720t SPL build Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 08/15] tegra20: remove timer_init from " Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 09/15] ARM: add tegra20 support to arm720t Allen Martin
2012-07-19 11:34 ` Simon Glass
2012-07-16 23:02 ` [U-Boot] [PATCH v7 10/15] tegra20: add u-boot-*-tegra.bin targets Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 11/15] tegra20: move SDRAM param save to later in boot Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 12/15] tegra20: enable SPL for tegra20 boards Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 13/15] arm: enable libgcc build for SPL Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 14/15] spl: fix SPL build of private libgcc Allen Martin
2012-07-16 23:02 ` [U-Boot] [PATCH v7 15/15] tegra20: Remove armv4t build flags Allen Martin
2012-07-17 19:32 ` [U-Boot] [PATCH v7 00/15] split tegra20 arm7 code into separate SPL Tom Warren
2012-07-19 0:02 ` Allen Martin [this message]
2012-07-19 15:37 ` Tom Warren
2012-07-19 20:09 ` Simon Glass
2012-07-23 20:38 ` Tom Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120719000200.GC20487@nvidia.com \
--to=amartin@nvidia.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox