[U-Boot] [PATCH V2 5/8] ARM: moved general function to arm/lib

Stefano Babic sbabic at denx.de
Sat Oct 15 13:12:43 CEST 2011


On 10/15/2011 12:15 PM, Albert ARIBAUD wrote:
> Hi Stefano,
> 

Hi Albert,

> 
> I agree with Wofgang that sdelay() is redundant wrt udelay() and has
> weaker semantics.
> 
> I'll add that sr32() is kind of not ARM specific, so I fail to see why
> it should move to generic ARM, and besides, it is a half-baked solution
> to the general problem of setting a bitfield in a register

Right, it is.

> 
> I conclude like Wolfgang that despite this code having gone through
> review unnoticed, it should not be promoted for wider use.

Understood.

> 
> If some functionality of these three functions is desirable and cannot
> be met with other means, please submit new code for general ARM
> inclusion (and maybe consider adapting armv7 to this new code).

I understand your points - and I agree with you, thanks for clarification.

Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================


More information about the U-Boot mailing list