From mboxrd@z Thu Jan  1 00:00:00 1970
From: Aneesh V <aneesh@ti.com>
Date: Tue, 05 Jul 2011 15:39:10 +0530
Subject: [U-Boot] [PATCH 3/6] arm: add __ilog2 function
In-Reply-To: <20110705095838.CD2A013D91D1@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>
Message-ID: <4E12E2C6.7020401@ti.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

Dear Wolfgang,

On Tuesday 05 July 2011 03:28 PM, Wolfgang Denk wrote:
> Dear Aneesh V,
>
> In message<4E129653.7050807@ti.com>  you wrote:
>>
>> I tried doing the same sometime back for the same need. I had to
>> abandon it because CLZ seems to be added only in ARMv5. And at least
>> one ARMv7 SoC uses -march=armv4 while building.
>
> Does this setting make any sense, or should we fix that?

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

Even if we could fix this, Rob is adding the API for all 'arm', which I
feel is not correct.

best regards,
Aneesh