[U-Boot] [PATCH v2] nand: Fix cache and memory inconsistent issue

Stefan Roese
2008-12-01T19:38:44Z

On Monday 01 December 2008, Scott Wood wrote:
> Please factor this out into arch code, and make it shareable with other
> NAND code (such as nand_boot.c).

Yes, please.

> > + /*
> > + * We need clean dcache and invalidate
> > + * to sync between icache and dcache
> > + * before jump to RAM. make sure all of
> > + * NAND data write to memory.
>
> "Clean d-cache and invalidate i-cache, to make sure that
> no stale data is executed."

And the function name is not perfectly fitting for my taste. Why not extract 2
functions, flush_dcache_range() and invalidate_icache_range(). This reminds
me that some architectures/platforms already have those functions defined
(PPC4xx at least does). But the 2nd parameter is not size but end ("stolen"
from the Linux kernel). Perhaps it makes sense to use these functions here as
well.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================