[U-Boot] [RFC PATCH v2] arm: cache_v7: fix dcache invalidation

From: Peng Fan <b51431@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH v2] arm: cache_v7: fix dcache invalidation
Date: Tue, 1 Sep 2015 09:55:39 +0800	[thread overview]
Message-ID: <20150901015535.GA10197@shlinux2> (raw)
In-Reply-To: <1440772931-4345-1-git-send-email-list-09_u-boot@tqsc.de>

On Fri, Aug 28, 2015 at 04:42:11PM +0200, Markus Niebel wrote:
>From: Markus Niebel <Markus.Niebel@tq-group.com>
>
>according to
>
>http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.faqs/ka14802.html
>
>and the clarification given by Marc Rutland of ARM
>
>data cache invalidation should go from outer to inner cache.
>This patch corrects the order in
>
>flush_dcache_range
>flush_dcache_all

Should be invalidate_dcache_range and invalidate_dcache_all, right?

Reviewed-by: Peng Fan <Peng.Fan@freescale.com>

Later, you should Cc maintainers.

>
>Signed-off-by: Markus Niebel <Markus.Niebel@tq-group.com>
>---
>Changes for v2:
>- delete non related stuff (thanks to Hannes Schmelzer)
>
> arch/arm/cpu/armv7/cache_v7.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/arch/arm/cpu/armv7/cache_v7.c b/arch/arm/cpu/armv7/cache_v7.c
>index a5aa4fa..81eb286 100644
>--- a/arch/arm/cpu/armv7/cache_v7.c
>+++ b/arch/arm/cpu/armv7/cache_v7.c
>@@ -235,9 +235,9 @@ static void v7_inval_tlb(void)
> 
> void invalidate_dcache_all(void)
> {
>-	v7_maint_dcache_all(ARMV7_DCACHE_INVAL_ALL);
>-
> 	v7_outer_cache_inval_all();
>+
>+	v7_maint_dcache_all(ARMV7_DCACHE_INVAL_ALL);
> }
> 
> /*
>@@ -257,9 +257,9 @@ void flush_dcache_all(void)
>  */
> void invalidate_dcache_range(unsigned long start, unsigned long stop)
> {
>-	v7_dcache_maint_range(start, stop, ARMV7_DCACHE_INVAL_RANGE);
>-
> 	v7_outer_cache_inval_range(start, stop);
>+
>+	v7_dcache_maint_range(start, stop, ARMV7_DCACHE_INVAL_RANGE);
> }
> 
> /*
>-- 
>2.3.0
>
>_______________________________________________
>U-Boot mailing list
>U-Boot at lists.denx.de
>http://lists.denx.de/mailman/listinfo/u-boot

-- 