[U-Boot] [PATCH] pxa: Disable dcache on palmld, palmtc, zipitz2

Marek Vasut marex at denx.de
Wed Oct 31 17:41:08 CET 2012


Dear Tom Rini,

> On 10/31/12 04:55, Marek Vasut wrote:
> > Dear Simon Glass,
> > 
> >> These platforms don't include dcache support. Define
> >> CONFIG_SYS_DCACHE_OFF so that functions don't try to call
> >> non-existent routines like flush_dcache_range().
> >> 
> >> Signed-off-by: Simon Glass <sjg at chromium.org>
> > 
> > Is that needed? Why not fix PXA by defining stub cache routines ?
> 
> Adding stubs sounds more like a work-around than disabling support
> that we won't have to me.

Yes, but then, the stubs will be optimized out.

I see using CONFIG_SYS_DCACHE_OFF being abused here. Every platform should 
implement at least cache operations stubs, then it'd be valid to use 
CONFIG_SYS_DCACHE_OFF to indicate the cache shall always be off. But the code 
would at least compile and work with CONFIG_SYS_DCACHE_OFF enabled or disabled.

We might eventually even be able to weed out this CONFIG_SYS_DCACHE_OFF option.

Best regards,
Marek Vasut


More information about the U-Boot mailing list