[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