
On Wed, Aug 29, 2012 at 09:55:17AM -0700, Tom Warren wrote:
Allen/Albert,
-----Original Message----- From: Allen Martin [mailto:amartin@nvidia.com] Sent: Tuesday, August 28, 2012 5:08 PM To: Tom Warren; swarren@wwwdotorg.org; sjg@chromium.org; thierry.reding@avionic-design.de; dev@lynxeye.de Cc: u-boot@lists.denx.de; Allen Martin Subject: [PATCH v10 00/16] 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-v10
Changes: v10:
- added fix to MAKEALL script so that it correctly parses new boards.cfg
I applied this to u-boot-tegra/master and pushed the new code upstream. The pull request remains the same (except for the inclusion of the MAKEALL patch, of course). I can send a new one if required - please let me know.
Currently running a ./MAKEALL arm - I assume it'll complete w/o errors (except for the ohci-hcd.c warnings I mentioned previously that are not due to this patch series).
Changing subject line to get Albert's attention
Thanks Tom. I traced down why "./MAKEALL arm" and "./MAKEALL -a arm" come up with a different list of boards. It's because the LIST_arm rule in MAKEALL which "./MAKEALL arm" uses is buggy and error prone. It's building all the Atmel boards twice and skipping a bunch of others like the arm720t, arm946es, and arm1176 boards.
I'm going to work on a patch to make LIST_arm use the same logic as "./MAKEALL -a arm" but in the mean time I strongly suggest using "./MAKEALL -a arm" since it generates the correct list of boards.
-Allen