[U-Boot] [v3] command/cache: Add flush command
York Sun
yorksun at freescale.com
Sat Apr 6 01:02:42 CEST 2013
On 04/05/2013 03:09 PM, Wolfgang Denk wrote:
> Dear York Sun,
>
> In message <1365195056-20188-1-git-send-email-yorksun at freescale.com> you wrote:
>> When we copy code/data to the main memory, we may need to flush the
>> cache if required by architecture. It uses the existing function
>> flush_cache. Syntax is
>>
>> flush <addr> <size>
>
> Plain "flush" is way too generic a name. I think we should make it
> clear from the command invocation that we are dealing with caches
> here.
>
> Actually I think we should not even use a new command for this - we
> already have the "dcahe" and "icache" commands for this purpose.
>
> What do you think about implementiung this as a subcommand to
> "dcache"? Something like:
>
> So far:
>
> dcache on
> dcache off
>
> adding new:
>
> dcache flush => flush all
> dcache flush <addr> <size> => flush range
>
> I think this makes more sense. Comments?
>
>
It would if the command only deals with dcache. This command flushes
dcache _and_ invalidates icache.
If "flush_cache" is acceptable, we can use v2. If not, please suggest
one. My candidates are "flushcache", "cacheflush".
York
More information about the U-Boot
mailing list