[U-Boot] [PATCH V3 1/2] Add env vars describing U-Boot target board

[Joe Hershberger]

Hi Tom,

On Wed, Jul 11, 2012 at 6:46 PM, Tom Warren <TWarren at nvidia.com> wrote:
> Joe,
>
>> -----Original Message-----
>> From: Joe Hershberger [mailto:joe.hershberger at gmail.com]
>> Sent: Wednesday, July 11, 2012 2:54 PM
>> To: Wolfgang Denk
>> Cc: Stephen Warren; u-boot at lists.denx.de; Stephen Warren; Tom Warren
>> Subject: Re: [U-Boot] [PATCH V3 1/2] Add env vars describing U-Boot target
>> board
>>
>> On Wed, Jul 11, 2012 at 2:33 AM, Wolfgang Denk <wd at denx.de> wrote:
>> > Dear Stephen Warren,
>> >
>> > In message <1337714515-6575-1-git-send-email-swarren at wwwdotorg.org> you
>> wrote:
>> >> From: Stephen Warren <swarren at nvidia.com>
>> >>
>> >> This can be useful for generic scripts. For example, rather than hard-
>> >> coding a script to ext2load tegra-harmony.dtb, it could load
>> >> ${soc}-${board}.dtb and hence not need adjustments to run on multiple
>> >> boards.
>> >>
>> >> Signed-off-by: Stephen Warren <swarren at nvidia.com>
>> >
>> > This patch breaks the BC3450 board:
>> >
>> > Configuring for BC3450 board...
>> >    text    data     bss     dec     hex filename
>> >  289722   20652  284640  595014   91446 /work/wd/tmp-ppc/u-boot
>> > cmd_bc3450.c:55:0: warning: "CONFIG_SYS_CPU" redefined [enabled by
>> default]
>> > /work/wd/tmp-ppc/include/config.h:3:0: note: this is the location of the
>> previous definition
>> >
>>
>> It also breaks a number of ARM boards...
>>
>> LOG/at91sam9263ek_norflash_boot.ERR:3:/home/joe/u-boot/include/config.h:5:0:
>> warning: "CONFIG_SYS_BOARD" redefined [enabled by default]
> I don't see any of these doing a MAKEALL -s at91 on either u-boot-tegra/master or u-boot-arm/master, with gcc 4.6.2 and 4.4.1.

I noticed them when building u-boot-net/next (which is based on
u-boot/master).  Building with Sourcery gcc 4.6.1.  MAKEALL -a arm

It may be caused by something else.  I just noticed it and it seemed related.

Cheers,
-Joe