From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hong Xu <hong.xu@atmel.com>
Date: Wed, 10 Aug 2011 14:41:58 +0800
Subject: [U-Boot] [PATCH v4] ARM926ejs: Add routines to invalidate
	D-Cache
In-Reply-To: <4E4226FE.4010805@aribaud.net>
References: <1312944565-3279-1-git-send-email-hong.xu@atmel.com>
	<201108100752.12813.marek.vasut@gmail.com>
	<4E422278.6030102@atmel.com> <4E4226FE.4010805@aribaud.net>
Message-ID: <4E422836.10104@atmel.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

Hi Albert,

On 08/10/2011 02:36 PM, Albert ARIBAUD wrote:
> Hi Hong Xu,
>
> Le 10/08/2011 08:17, Hong Xu a ?crit :
>
>> There are some ARM926 specific code in arch/arm/lib/cache.c; So I also
>> put the stuff there. ;-) I think Albert Aribaud or the original
>> contributor of cache part shall have clearer view.So, I'll keep neutral
>> to hear more ideas.
>
> Basically, cache operations are CP15 commands which are defined for each
> ARM architecture, not for each ISA, so Marek is right about the best
> place for this being in arm926ejs. Actually, I think
> arch/arm/lib/cache.c should only contain the weak defaults (i.e., no
> real cache action) and each architecture should provide overrides to the
> defaults. Currently this is almost the case, with (apart from arm926ejs
> which you're already touching) only one arm1136 specific implementation
> to move.
>
> So please move the arm926ejs specific implementations to
> arch/arm/cpu/arm926ejs/cache.c as suggested by Marek, keeping only the
> weak default in arch/arm/lib/cache.c.

Ok, I'll manage a basic version for review.

Thanks

BR,
Eric

>
> Amicalement,