Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-03-03 Thread Mark Rutland
On Thu, Feb 26, 2015 at 03:06:10PM +, FengHua wrote: > > hi Mark, Hi, >You did very detailed analysis of the cache beheaviour. Yes, this > patch is not perfect. > But it did fix the actually existed bug. I will try to describe it more > clearly in the following. While this may app

Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-02-26 Thread FengHua
2015-02-11 23:51:15 (Wednesday) > To: FengHua > Cc: "u-boot@lists.denx.de" , "albert.u.boot" > > Subject: Re: Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable() > > On Wed, Feb 11, 2015 at 03:26:06AM +, FengHua wrote: > > > > hi Mark, &

Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-02-11 Thread Mark Rutland
ot; > > Cc: "u-boot@lists.denx.de" > > Subject: Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable() > > > > On Mon, Feb 09, 2015 at 08:51:59AM +, feng...@phytium.com.cn wrote: > > > From: David Feng > > > > > > The cach

Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-02-10 Thread FengHua
hi Mark, Thank you review this patch. > -Original Messages- > From: "Mark Rutland" > Sent Time: 2015-02-09 19:05:54 (Monday) > To: "feng...@phytium.com.cn" > Cc: "u-boot@lists.denx.de" > Subject: Re: [U-Boot] [PATCH] ARMv8: Bug fix

Re: [U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-02-09 Thread Mark Rutland
On Mon, Feb 09, 2015 at 08:51:59AM +, feng...@phytium.com.cn wrote: > From: David Feng > > The cache disable operation shoud be performed after flush_dcache_all(). > If cache disable operation is performed before > flush_dcache_all(), flush_dcache_all() store data directly to memory > and may

[U-Boot] [PATCH] ARMv8: Bug fix of dcache_disable()

2015-02-09 Thread fenghua
From: David Feng The cache disable operation shoud be performed after flush_dcache_all(). If cache disable operation is performed before flush_dcache_all(), flush_dcache_all() store data directly to memory and may be overrided by data copy in cache. Signed-off-by: David Feng --- arch/arm/cpu/a