[U-Boot] [PATCH] OMAP3: Use sdelay from arch/arm/cpu/armv7/syslib.c instead of cloning that.

Dirk Behme dirk.behme at googlemail.com
Sun Dec 19 08:54:46 CET 2010


On 18.12.2010 13:24, Alexander Holler wrote:
> There is no need to have such a function twice.
>
> Signed-off-by: Alexander Holler<holler at ahsoftware.de>
> ---
>   arch/arm/cpu/armv7/omap3/board.c |   12 +-----------
>   1 files changed, 1 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/omap3/board.c b/arch/arm/cpu/armv7/omap3/board.c
> index 6c2a132..0e62626 100644
> --- a/arch/arm/cpu/armv7/omap3/board.c
> +++ b/arch/arm/cpu/armv7/omap3/board.c
> @@ -41,16 +41,6 @@
>   extern omap3_sysinfo sysinfo;
>
>   /******************************************************************************
> - * Routine: delay
> - * Description: spinning delay to use before udelay works
> - *****************************************************************************/
> -static inline void delay(unsigned long loops)
> -{
> -	__asm__ volatile ("1:\n" "subs %0, %1, #1\n"
> -			  "bne 1b":"=r" (loops):"0"(loops));
> -}
> -
> -/******************************************************************************
>    * Routine: secure_unlock
>    * Description: Setup security registers for access
>    *              (GP Device only)
> @@ -189,7 +179,7 @@ void s_init(void)
>   		setup_auxcr();
>
>   	set_muxconf_regs();
> -	delay(100);
> +	sdelay(100);
>
>   	prcm_init();
>

Looking at the implementation of sdelay(), yes, it seems it's just an 
identical copy of delay(). So:

Acked-by: Dirk Behme <dirk.behme at googlemail.com>

Thanks

Dirk


More information about the U-Boot mailing list