[U-Boot] [v3] command/cache: Add flush command
Wolfgang Denk
wd at denx.de
Sat Apr 6 09:01:50 CEST 2013
Dear York Sun,
In message <515F5812.8030008 at freescale.com> you wrote:
>
> > 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.
Then the name "flush" is even more a bad choice.
> If "flush_cache" is acceptable, we can use v2. If not, please suggest
> one. My candidates are "flushcache", "cacheflush".
Can we not split this into:
dcache flush
icache invalidate
? This would make clear what's happening.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
READ THIS BEFORE OPENING PACKAGE: According to Certain Suggested Ver-
sions of the Grand Unified Theory, the Primary Particles Constituting
this Product May Decay to Nothingness Within the Next Four Hundred
Million Years.
More information about the U-Boot
mailing list