diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2019-09-09 10:21:38 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2019-09-09 10:26:07 +0200 |
commit | 6b3dc4abd884e7e3e1ac734d23ebbbfab1d44171 (patch) | |
tree | 44f5ea07ccb96e56a1c0a5ce92173ce5d4d70b1b /arch/arm/cpu | |
parent | 971307cf0132d02b3d361117c2b1f90893404eab (diff) | |
download | barebox-6b3dc4abd884e7e3e1ac734d23ebbbfab1d44171.tar.gz barebox-6b3dc4abd884e7e3e1ac734d23ebbbfab1d44171.tar.xz |
ARM: Cleanup stack offset cargo cult
Most callers of arm_setup_stack substract a fixed offset of 8, 12 or 16
bytes from the stack top. This is unnecessary as on ARM we have a stack
that decrements before storing values. Substracting this offset probably
goes back to the U-Boot version we forked from. Stop this now.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'arch/arm/cpu')
-rw-r--r-- | arch/arm/cpu/entry.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/cpu/entry.c b/arch/arm/cpu/entry.c index b48c1ca11d..30df95f078 100644 --- a/arch/arm/cpu/entry.c +++ b/arch/arm/cpu/entry.c @@ -27,7 +27,7 @@ void NAKED __noreturn barebox_arm_entry(unsigned long membase, unsigned long memsize, void *boarddata) { - arm_setup_stack(arm_mem_stack_top(membase, membase + memsize) - 16); + arm_setup_stack(arm_mem_stack_top(membase, membase + memsize)); arm_early_mmu_cache_invalidate(); if (IS_ENABLED(CONFIG_PBL_MULTI_IMAGES)) |