From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Tue, 05 Jul 2011 13:56:22 +0200
Subject: [U-Boot] [PATCH 3/6] arm: add __ilog2 function
In-Reply-To: <20110705111725.80E9D15794D3@gemini.denx.de>
References: <1309275583-11763-1-git-send-email-robherring2@gmail.com>	<1309275583-11763-4-git-send-email-robherring2@gmail.com>	<4E129653.7050807@ti.com>	<20110705095838.CD2A013D91D1@gemini.denx.de>	<4E12E2C6.7020401@ti.com>
	<20110705111725.80E9D15794D3@gemini.denx.de>
Message-ID: <4E12FBE6.3000600@aribaud.net>
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

Le 05/07/2011 13:17, Wolfgang Denk a ?crit :
> Dear Aneesh V,
>
> In message<4E12E2C6.7020401@ti.com>  you wrote:
>>
>> Looks like it makes sense. Here is what the comment says in
>> arch/arm/cpu/armv7/tegra2/config.mk
>>
>> # Use ARMv4 for Tegra2 - initial code runs on the AVP, which is an ARM7TDI.
>> PLATFORM_CPPFLAGS += -march=armv4
>
> I'm not sure what "initial code" here means - all of U-Boot?
>
>> Even if we could fix this, Rob is adding the API for all 'arm', which I
>> feel is not correct.
>
> Agreed.  But then, it's the overwhelming majority that could use this
> code.

Maybe we could condition this __ilog2 definition on whether we build for 
armv4 (and below) or armv5t (or above)?

> Best regards,
>
> Wolfgang Denk

Amicalement,
-- 
Albert.