On Thu, Dec 22, 2011 at 10:09:30AM +0100, Michal Simek wrote:
> Stefan Kristiansson wrote:
> >+
> >+void dcache_enable(void)
> >+{
> >+ mtspr(SPR_SR, mfspr(SPR_SR) | SPR_SR_DCE);
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
> >+ asm volatile("l.nop");
>
> This is interesting. Are there 8 nops?
> Is there any reason for that? Is it just any waiting?
> If yes, maybe it will be worth to add any comment.
>
Yes, the number of nops are kind of arbitrary,
they are there to add some headroom for the cache
to startup without any load/stores in flight.
But you are right, a comment about this might be in place.
Stefan
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot
