Dear sun york-R58495,

In message 
<c707e9f4d8007146bf8dc1424b113ac70b3a4...@039-sn2mpn1-012.039d.mgd.msft.net> 
you wrote:
> 
> > Can we not split this into:
> > 
> >     dcache flush
> >     icache invalidate
> > 
> > ?  This would make clear what's happening.
> 
> 
> The idea is to reuse existing code with minimum addition. For the applicati
> ons concerned, these two steps are both needed. Splitting them doesn't make
>  things easier.
Reusing code is a Good Thing, but not when it comes at the cost of
obfucating what the code actually does.

> If I have to use existing command, I'd rather to put these two steps under
> icache invalide <addr> <size>.

No, this is not acceptable.  The "icache" command deals with the IC
only, it must not meddle with the data cache (like flushing it).

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: w...@denx.de
Death, when unnecessary, is a tragic thing.
        -- Flint, "Requiem for Methuselah", stardate 5843.7
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to