
Dear Rafal,
in message 477CB941.2020001@semihalf.com you wrote:
Indeed, in certain cases it does actually (e.g. 86xx), but it's not uniform unfortunatelly. In my case (85xx) dcache_disable() does not flush the cache before disabling.
I consider this a bug in the 85xx code, then. The bug should be fixed, and not worked around it.
Yeah, after a second thought I tend to agree. Maybe the way to go is doing data cache flush from within dcache_disable() properly i.e. bring it in for arch variations that don't do it currently like 85xx... Actually to confirm my observations I tested a working patch that flushes d-cache at cache_disable() just like 86xx and it works for me, this is: my problems disappear. Do you think this is a better option?
Yes, indeed.
Best regards,
Wolfgang Denk