diff options
| author | Vitaly Kuzmichev <vkuzmichev@mvista.com> | 2010-06-15 22:18:11 +0400 |
|---|---|---|
| committer | Wolfgang Denk <wd@denx.de> | 2010-06-22 22:15:07 +0200 |
| commit | a71da1b6c96205549ca2e7cf991e2340181bbfcf (patch) | |
| tree | 4956c3b08860ad3467ac68d51ec2d2c1f190ab6d /arch/arm/cpu/arm926ejs | |
| parent | 96b35730a43160afa025ac405961737dbb0682b4 (diff) | |
| download | u-boot-imx-a71da1b6c96205549ca2e7cf991e2340181bbfcf.zip u-boot-imx-a71da1b6c96205549ca2e7cf991e2340181bbfcf.tar.gz u-boot-imx-a71da1b6c96205549ca2e7cf991e2340181bbfcf.tar.bz2 | |
ARM: Align stack to 8 bytes
The ARM ABI requires that the stack be aligned to 8 bytes as it is noted
in Procedure Call Standard for the ARM Architecture:
http://infocenter.arm.com/help/topic/com.arm.doc.ihi0042d/index.html
Unaligned SP also causes the problem with variable-length arrays
allocation when VLA address becomes less than stack pointer during
aligning of this address, so the next 'push' in the stack overwrites
first 4 bytes of VLA.
Signed-off-by: Vitaly Kuzmichev <vkuzmichev@mvista.com>
Tested on tx25(mx25), imx27lite(mx27), qong(mx31) and trab(s3c2400)
Tested-by: Wolfgang Denk <wd@denx.de>
Diffstat (limited to 'arch/arm/cpu/arm926ejs')
| -rw-r--r-- | arch/arm/cpu/arm926ejs/start.S | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/cpu/arm926ejs/start.S b/arch/arm/cpu/arm926ejs/start.S index 3b81151..cf40ce1 100644 --- a/arch/arm/cpu/arm926ejs/start.S +++ b/arch/arm/cpu/arm926ejs/start.S @@ -196,7 +196,7 @@ stack_setup: #endif #endif /* CONFIG_PRELOADER */ sub sp, r0, #12 /* leave 3 words for abort-stack */ - bic sp, r0, #7 /* 8-byte align stack for ABI compliance */ + bic sp, sp, #7 /* 8-byte alignment for ABI compliance */ clear_bss: ldr r0, _bss_start /* find start of bss segment */ |