From mboxrd@z Thu Jan  1 00:00:00 1970
From: Rob Herring <robherring2@gmail.com>
Date: Sun, 21 Aug 2011 14:28:29 -0500
Subject: [U-Boot] [PATCH v5] ARM: Add Calxeda Highbank platform
In-Reply-To: <4E5118C2.7010404@aribaud.net>
References: <1313932575-31456-1-git-send-email-robherring2@gmail.com>
	<4E5118C2.7010404@aribaud.net>
Message-ID: <4E515C5D.3030902@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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@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@calxeda.com>
>> Signed-off-by: Rob Herring<rob.herring@calxeda.com>
>> Cc: Albert ARIBAUD<albert.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 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.

Rob