[U-Boot] [PATCH 00/32] Initial sparse fix series

Tom Rini
2012-10-25T19:28:26Z

On Thu, Oct 25, 2012 at 01:59:45PM -0500, Kim Phillips wrote:
> On Thu, 25 Oct 2012 10:46:54 -0700
> Tom Rini <trini@ti.com> wrote:
>
> > On Tue, Oct 16, 2012 at 07:28:16PM -0500, Kim Phillips wrote:
> >
> > > This 32-patch series only begins to address making u-boot source more
> > > 'sparseable,' or sparse-clean, ultimately to catch type, address space,
> > > and endianness mismatches and generally improve code quality. E.g., in this
> > > initial dose whose main purpose is to reduce the output volume to workable
> > > levels, a couple of endianness bugs are found and fixed in
> > > of_bus_default_translate() and fdt_get_base_address(). See [PATCH 14/32]
> > > common/fdt_support.c: sparse fixes.
> >
> > I want to repeat myself that I want to see sparse fixes come in. That
> > said, I expect that for v2 you will have done MAKEALL -a $arch for every
> > arch there's an ELDK 5.2.x toolchain for, at least, and there's no new
>
> so power, arm, and mips.

Correct.

> > problems popping up. If you like I can pastebin my MAKEALL wrapper that
> > lets you tell it things like --arch arm --eldk-521 and it sets up the
> > environment for the toolchain.
>
> please do.

http://pastebin.com/rkRSL2d3 has it now. It's a little clumsy at times,
for example for not-arm (biased, sorry) you need to do uboot-build.sh
--arch powerpc --eldk-521 --cpu mpc85xx so that it sets up for arch/powerpc,
and then the toolchain and then re-sets itself for cpu/mpc85xx.
arch/cpu/soc default to NCPUS=1 NBUILDS=`grep -c processor
/proc/cpuinfo` and single board targets default to letting MAKEALL guess
right. I'm pretty sure that even on the bigger boxes this is still
right.

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121025/91db66c3/attachment.pgp>