From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Thu, 28 May 2009 21:28:36 +0200
Subject: [U-Boot] [PATCH] The cache flush using coprocessor must be
	in	lib_arm/cache-cp15.c
In-Reply-To: <b64afca20905280151p4ed19e2bq90b7b804356fc0ed@mail.gmail.com>
References: <b64afca20905280151p4ed19e2bq90b7b804356fc0ed@mail.gmail.com>
Message-ID: <20090528192836.GA1802@game.jcrosoft.org>
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

On 17:51 Thu 28 May     , Kim, Heung Jun wrote:
> Hi? I'm about to suggest one more thing related to "change cpu.c under
> cpu/arm_cortexa8 dir to common code.".
> 
> asm ("mcr p15, 0, %0, c7, c5, 0": :"r" (0));
> 
> It's the common feature all over the arm core, not only arm cortex A8.
> The common cache function is defined in the lib_arm/cache-cp15.c as you
> know. So, It's seems the better method that cache_flush() is moved to
> lib_arm/cache-cp15.c
NACK

to boot linux we are suppose to clean both cache I and DCache

the I/DCache flush is armv and arch dependant (example ARM7TDMI, armv4wt, armv3,
armv7, etc...)

so for now please let it in cpu/$(ARCH)/cpu.c

Best Regards,
J.