From de01136484c58d13457ccf1e42fdb2310f3cbe65 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ky=C3=B6sti=20M=C3=A4lkki?= Date: Thu, 17 Nov 2016 22:39:29 +0200 Subject: intel post-car: Increase stacktop alignment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Align top of stack to 8 bytes, value documented as FSP1.1 requirement. Also fix some cases of uintptr_t casted to unsigned long. Change-Id: I5bbd100eeb673417da205a2c2c3410fef1af61f0 Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/17461 Reviewed-by: Aaron Durbin Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel --- src/lib/romstage_stack.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/lib') diff --git a/src/lib/romstage_stack.c b/src/lib/romstage_stack.c index fde9d1b9b2..ed6f09b4cb 100644 --- a/src/lib/romstage_stack.c +++ b/src/lib/romstage_stack.c @@ -44,5 +44,8 @@ uintptr_t romstage_ram_stack_top(void) uintptr_t stack_top = romstage_ram_stack_base(ROMSTAGE_RAM_STACK_SIZE, ROMSTAGE_STACK_CBMEM); stack_top += ROMSTAGE_RAM_STACK_SIZE; + + /* Make it aligned to a 8-byte boundary. */ + stack_top = ALIGN_DOWN(stack_top, 8); return stack_top; } -- cgit v1.2.3