[U-Boot] ./MAKEALL arm is buggy

Allen Martin
2012-08-29T18:33:06Z

On Wed, Aug 29, 2012 at 09:55:17AM -0700, Tom Warren wrote:
> Allen/Albert,
>
> > -----Original Message-----
> > From: Allen Martin [mailto:amartin at nvidia.com]
> > Sent: Tuesday, August 28, 2012 5:08 PM
> > To: Tom Warren; swarren at wwwdotorg.org; sjg at chromium.org;
> > thierry.reding at avionic-design.de; dev at lynxeye.de
> > Cc: u-boot at 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
--
nvpublic