[U-Boot] [PATCH v5] ARM: Add Calxeda Highbank platform

[Rob Herring]

Albert,

On 08/21/2011 02:28 PM, Rob Herring wrote:
> On 08/21/2011 09:40 AM, Albert ARIBAUD wrote:
>> Hi Rob,
>>
>> Le 21/08/2011 15:16, Rob Herring a écrit :
>>> From: Rob Herring<rob.herring at calxeda.com>
>>>
>>> Add basic support for Calxeda Highbank platform. Only minimal support
>>> with
>>> serial and SATA are included.
>>>
>>> Signed-off-by: Jason Hobbs<jason.hobbs at calxeda.com>
>>> Signed-off-by: Rob Herring<rob.herring at calxeda.com>
>>> Cc: Albert ARIBAUD<albert.u.boot at aribaud.net>
>>> ---
>>
>> This applies to u-boot-arm/master, with three empty lines at end of new
>> files removed, but the compilation with ELDK 4.2 fails with:
>>
>> uboot at lilith:~/src/u-boot-arm$ ./MAKEALL highbank
>> Configuring for highbank board...
>> include/common.h:0: error: bad value (armv7-a) for -march= switch
>> include/common.h:0: error: bad value (armv7-a) for -march= switch
>> lib/asm-offsets.c:1: error: bad value (armv7-a) for -march= switch
>> make: *** [lib/asm-offsets.s] Erreur 1
>> arm-linux-size: './u-boot': No such file
>>
>> --------------------- SUMMARY ----------------------------
>> Boards compiled: 1
>> Boards with warnings or errors: 1 ( highbank )
>> ----------------------------------------------------------
>>
>> Seems like ELDK 4.2 does not know armv7-a. Is it a requirement that this
>> board build with this march flag?
>>
> 
> With device tree, there are lots of endian conversions. With v7, those
> conversions are a single inline "rev" instruction vs. 10-20 assembly
> instructions. Since there is no state maintained in the fdt code, the
> conversions are done over and over.
> 
> So there's not a hard requirement, but it is something I will always set
> and I imagine others with v7 platforms will want to also. I think Tegra
> prevents doing this change globally.
> 

What do you want to do here?

arch/arm/cpu/armv7/highbank/config.mk can simply be deleted if you don't
want to compile with armv7-a.

Rob