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 Herringrob.herring@calxeda.com
Add basic support for Calxeda Highbank platform. Only minimal support with serial and SATA are included.
Signed-off-by: Jason Hobbsjason.hobbs@calxeda.com Signed-off-by: Rob Herringrob.herring@calxeda.com Cc: Albert ARIBAUDalbert.u.boot@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@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