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
