Am Mo., 10. Sep. 2018 um 07:34 Uhr schrieb Stefan Roese <s...@denx.de>:
>
> Hi Daniel,
>
> On 07.09.2018 19:02, Daniel Schwierzeck wrote:
> > Those functions are not needed during cache init and can be
> > implemented in C. Only support the safe disabling of caches when
> > this is required for booting an OS. Reenabling caches is much
> > harder to implement if an optional coherency manager must be
> > supported. As there is no real use-case anyway, dcache_enable
> > is implemented with an error message.
> >
> > Signed-off-by: Daniel Schwierzeck <daniel.schwierz...@gmail.com>
>
> Many thanks for working on this. I'll be doing some more test
> later this week using this series. One comment though directly:
>
> A few weeks ago I wanted to enable the "cache" cmd (CMD_CACHE)
> on my MT7688 MIPS platform. Unfortunately some I-Cache functions
> are missing for this:
>
> building ...
> cmd/built-in.o: In function `do_icache':
> cmd/cache.c:(.text.do_icache+0x48): undefined reference to `icache_disable'
> cmd/cache.c:(.text.do_icache+0x68): undefined reference to `icache_enable'
> cmd/cache.c:(.text.do_icache+0x88): undefined reference to `icache_status'
> Makefile:1350: recipe for target 'u-boot' failed
>
> Would it be possible to add those functions as well to this
> series (or some follow-up patch)? Using the cache cmd can be
> quite helpful while testing / debugging.
>

The problem is that you can't independently disable I-Cache and
D-Cache on MIPS. I could add them as dummy functions or with the same
implementation as dcache_*

-- 
- Daniel
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to