[U-Boot] [PATCH 3/6] arm: add __ilog2 function

Albert ARIBAUD albert.u.boot at aribaud.net
Tue Jul 5 13:56:22 CEST 2011


Le 05/07/2011 13:17, Wolfgang Denk a écrit :
> Dear Aneesh V,
>
> In message<4E12E2C6.7020401 at 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.


More information about the U-Boot mailing list