Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-27 Thread Stefan Roese
Hi! (added a few more people to the Cc list) On 17.03.2015 10:00, Holger Brunck wrote: Hi Tom, On 03/13/2015 03:34 PM, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-27 Thread Nitin Garg
Hi Stefan, On 03/27/2015 08:18 AM, Stefan Roese wrote: Hi! (added a few more people to the Cc list) On 17.03.2015 10:00, Holger Brunck wrote: Hi Tom, On 03/13/2015 03:34 PM, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-17 Thread Holger Brunck
Hi Tom, On 03/13/2015 03:34 PM, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the updated bootcounter may not be saved to its location. This was detected on an iMX.6 platform

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-16 Thread York Sun
Tom, On 03/15/2015 11:30 AM, Tom Rini wrote: ... Add a few more PowerPC people. On Fri, Mar 13, 2015 at 10:34:03AM -0400, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-16 Thread Tom Rini
On Mon, Mar 16, 2015 at 08:57:02AM -0700, York Sun wrote: Tom, On 03/15/2015 11:30 AM, Tom Rini wrote: ... Add a few more PowerPC people. On Fri, Mar 13, 2015 at 10:34:03AM -0400, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-16 Thread York Sun
On 03/16/2015 10:05 AM, Tom Rini wrote: 8xx and 83xx just need the dummy files copied over. OK, I went and poked at going one direction on this and then dug into the higher level problem more. PowerPC _needs_ the current kernel's arch/powerpc/kernel/misc_32.S relevant cache functions

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-16 Thread Tom Rini
On Mon, Mar 16, 2015 at 10:11:17AM -0700, York Sun wrote: On 03/16/2015 10:05 AM, Tom Rini wrote: 8xx and 83xx just need the dummy files copied over. OK, I went and poked at going one direction on this and then dug into the higher level problem more. PowerPC _needs_ the current

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-15 Thread Tom Rini
... Add a few more PowerPC people. On Fri, Mar 13, 2015 at 10:34:03AM -0400, Tom Rini wrote: On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the updated bootcounter may not be saved to

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-13 Thread Tom Rini
On Fri, Mar 13, 2015 at 09:48:56AM -0400, Tom Rini wrote: On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the updated bootcounter may not be saved to its location. This was detected on an iMX.6 platform using the OCRAM (internal SRAM) as

Re: [U-Boot] bootcount: Add dcache flush to bootcount_store()

2015-03-13 Thread Tom Rini
On Wed, Mar 11, 2015 at 09:51:38AM +0100, Stefan Roese wrote: Without this dcache_flush the updated bootcounter may not be saved to its location. This was detected on an iMX.6 platform using the OCRAM (internal SRAM) as bootcounter storage area. And issuing reset from within U-Boot cause