[U-Boot] [PATCH] arch/arm/cpu/arm_cortexa8/omap3/cache.S: make build with older tools

Wolfgang Denk wd at denx.de
Fri Jun 18 17:13:26 CEST 2010


Dear Albin Tonnerre,

In message <AANLkTimaci9vYr8Hpnq99wnJzowKNIqgY7oXK_Vr0g6W at mail.gmail.com> you wrote:
> 
> >   l2_cache_enable:
> > -          push      {r0, r1, r2, lr}
> > +          stmfd    r13!, {r0, r1, r2, lr}
> >            @ ES2 onwards we can disable/enable L2 ourselv=
> es
> >            bl         get_cpu_rev
> >            cmp       r0, #CPU_3XX_ES20
> > @@ -157,11 +157,11 @@ l2_cache_enable_EARLIER_THAN_ES2:
> >            mov       ip, r3
> >            str       r3, [sp, #4]
> >   l2_cache_enable_END:
> > -          pop       {r1, r2, r3, pc}
> > +          ldmfd    r13!, {r1, r2, r3, pc}
> 
> For clarity's sake, it might be worth using sp instead of r13

Agreed, but for the sake of consisitency (all other similar places in
this file use r13 as well) I decided to stick with the existing
convention.

Maybe we should submit an additional cleanp-patch that changes r13
into sp when used like this? (or always?)

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
An armed society is a polite society.


More information about the U-Boot mailing list