Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-20 Thread Siva Durga Prasad Paladugu
Hi Mark, > -Original Message- > From: Mark Rutland [mailto:mark.rutl...@arm.com] > Sent: Monday, April 20, 2015 3:48 PM > To: Siva Durga Prasad Paladugu > Cc: Michal Simek; u-boot@lists.denx.de; Tom Rini; Varun Sethi; Arnab Basu; > York Sun > Subject: Re: [U-Bo

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-20 Thread Mark Rutland
> > > Thanks for explanation. > > > So in that case, the flushing of the required stack or any other data > > > which needs to be flushed should be part of board specific. Am I > > > correct? > > > > It could be done in generic code, assuming we know the bounds of memory > > which will be used, bec

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-17 Thread Siva Durga Prasad Paladugu
Hi Mark, > -Original Message- > From: Mark Rutland [mailto:mark.rutl...@arm.com] > Sent: Friday, April 17, 2015 3:36 PM > To: Siva Durga Prasad Paladugu > Cc: Michal Simek; u-boot@lists.denx.de; Tom Rini; Varun Sethi; Arnab Basu; > York Sun > Subject: Re: [U-Bo

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-17 Thread Mark Rutland
> > > Now in the flush_dcache_all we are invoking the actual asm call to > > > flush dcache which may wipeout the stored return value in stack with > > > cahe contents(main memory). Hence the return from the flush_dcahe_all > > > will fail. > > > > > > To confirm this I modified the dcache_disable

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-16 Thread Siva Durga Prasad Paladugu
Wednesday, April 15, 2015 6:41 PM > > > To: Michal Simek > > > Cc: u-boot@lists.denx.de; Tom Rini; Siva Durga Prasad Paladugu; > > > Varun Sethi; Arnab Basu; York Sun > > > Subject: Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache > > > aft

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-16 Thread Mark Rutland
denx.de; Tom Rini; Siva Durga Prasad Paladugu; Varun Sethi; > > Arnab Basu; York Sun > > Subject: Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush > > > > On Wed, Apr 15, 2015 at 12:33:00PM +0100, Michal Simek wrote: > > > From: Siva Durga Prasad

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-15 Thread Siva Durga Prasad Paladugu
Hi Mark. > -Original Message- > From: Mark Rutland [mailto:mark.rutl...@arm.com] > Sent: Wednesday, April 15, 2015 6:41 PM > To: Michal Simek > Cc: u-boot@lists.denx.de; Tom Rini; Siva Durga Prasad Paladugu; Varun Sethi; > Arnab Basu; York Sun > Subject: Re: [U-Bo

Re: [U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-15 Thread Mark Rutland
On Wed, Apr 15, 2015 at 12:33:00PM +0100, Michal Simek wrote: > From: Siva Durga Prasad Paladugu > > Always disable dcache after the flush operation > The following sequence is advisable while disabling d-cache: > 1. disable_dcache() - flushes and disables d-cache > 2. invalidate_dcache_all() - i

[U-Boot] [PATCH 1/2] armv8: caches: Disable dcache after flush

2015-04-15 Thread Michal Simek
From: Siva Durga Prasad Paladugu Always disable dcache after the flush operation The following sequence is advisable while disabling d-cache: 1. disable_dcache() - flushes and disables d-cache 2. invalidate_dcache_all() - invalid any entry that came to the cache in the short period after the c