From 89be154f0fffb3fcf06ce3ff9bb5f8c1edd0e02b Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Thu, 18 Dec 2014 19:24:48 -0800 Subject: Kconfig: Fix incorrect CONFIG_STACK_SIZE values for X86 and ARM64 Commit 54229a7 (arm: Fix checkstack() to use correct stack size) didn't quite hit the mark. Due to the crazy way our Kconfig includes work, It accidentally set CONFIG_STACK_SIZE to 0 even on architectures that need it. This patch fixes the issue by moving everything back to a single entry in src/Kconfig, making sure we end up with the intended numbers on all architectures. BRANCH=None BUG=chrome-os-partner:34750 TEST=Built for Pinky, Urara, Falco and Ryu. Confirmed that the generated .config contained CONFIG_STACK_SIZE=0x0 for the former two, and CONFIG_STACK_SIZE=0x1000 for the latter. Original-Change-Id: Ib18561925aafe7c74e6c4f0b10b55000a785e144 Original-Signed-off-by: Julius Werner Original-Reviewed-on: https://chromium-review.googlesource.com/236753 Original-Reviewed-by: David Hendricks (cherry picked from commit c64b127e163f98162f3f7195b6ed09bd5a4b77c4) Signed-off-by: Aaron Durbin Change-Id: I2c747b04760bc97f43523596640bfb15317e5730 Reviewed-on: http://review.coreboot.org/9696 Reviewed-by: David Hendricks Reviewed-by: Edward O'Callaghan Tested-by: build bot (Jenkins) --- src/arch/arm/Kconfig | 5 ----- 1 file changed, 5 deletions(-) (limited to 'src/arch/arm/Kconfig') diff --git a/src/arch/arm/Kconfig b/src/arch/arm/Kconfig index 6bef91ce77..2499d2d075 100644 --- a/src/arch/arm/Kconfig +++ b/src/arch/arm/Kconfig @@ -31,8 +31,3 @@ config ARM_BOOTBLOCK_CUSTOM config ARM_LPAE bool default n - -# Mark SMP stack size as 0 since we keep using SRAM stack throughout ramstage. -config STACK_SIZE - hex - default 0x0 -- cgit v1.2.3