armv5te: make 'ret lr' produce iinterworking 'bx lr'
authorAlbert ARIBAUD <albert.u.boot@aribaud.net>
Mon, 27 Feb 2017 19:19:07 +0000 (20:19 +0100)
committerTom Rini <trini@konsulko.com>
Thu, 2 Mar 2017 02:28:31 +0000 (21:28 -0500)
Current ARM assembler helper for the 'return to caller' pseudo-instruction
turns 'ret lr' into 'mov pc, lr' for ARMv5TE. This causes the core to remain
in its current ARM state even when the routine doing the 'ret' was called
from Thumb-1 state, triggering an undefined instruction exception.

This causes early run-time failures in all boards compiled using the Thumb-1
instruction set (for instance the Open-RD family).

ARMv5TE supports 'bx lr' which properly implements interworking and thus
correctly returns to Thumb-1 state from ARM state.

This change makes 'ret lr' turn into 'bx lr' for ARMv5TE.

Signed-off-by: Albert ARIBAUD <albert.u.boot@aribaud.net>
arch/arm/include/asm/assembler.h

index ae1e42fc06e865b7539061c7479915163467b0f2..c56daf2a1f69b2a64f804642128bf08bcd01f5bf 100644 (file)
@@ -59,7 +59,7 @@
 
        .irp    c,,eq,ne,cs,cc,mi,pl,vs,vc,hi,ls,ge,lt,gt,le,hs,lo
        .macro  ret\c, reg
-#if defined(__ARM_ARCH_5E__) || defined(__ARM_ARCH_5TE__)
+#if defined(__ARM_ARCH_5E__)
        mov\c   pc, \reg
 #else
        .ifeqs  "\reg", "lr"