5 Jul
2011
5 Jul
'11
1:56 p.m.
Le 05/07/2011 13:17, Wolfgang Denk a écrit :
Dear Aneesh V,
In message4E12E2C6.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.