[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