On 05/17/2016 12:37 AM, Masahiro Yamada wrote:
> __asm_dcache_all can directly return to the caller of
> __asm_{flush,invalidate}_dcache_all.
> 
> We do not have to waste x16 register here.
> 
> Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com>
> ---
> 
>  arch/arm/cpu/armv8/cache.S | 10 ++--------
>  1 file changed, 2 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/cpu/armv8/cache.S b/arch/arm/cpu/armv8/cache.S
> index a9f4fec..1c71a2f 100644
> --- a/arch/arm/cpu/armv8/cache.S
> +++ b/arch/arm/cpu/armv8/cache.S
> @@ -104,19 +104,13 @@ finished:
>  ENDPROC(__asm_dcache_all)
>  
>  ENTRY(__asm_flush_dcache_all)
> -     mov     x16, lr
>       mov     x0, #0
> -     bl      __asm_dcache_all
> -     mov     lr, x16
> -     ret
> +     b       __asm_dcache_all
>  ENDPROC(__asm_flush_dcache_all)
>  
>  ENTRY(__asm_invalidate_dcache_all)
> -     mov     x16, lr
>       mov     x0, #0x1
> -     bl      __asm_dcache_all
> -     mov     lr, x16
> -     ret
> +     b       __asm_dcache_all
>  ENDPROC(__asm_invalidate_dcache_all)
>  
>  /*
> 

It is a smart solution.
Reviewed-by: York Sun <york....@nxp.com>

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to