diff options
| author | Aaron Durbin <adurbin@adurbin.bld.corp.google.com> | 2017-07-11 17:29:02 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-07-13 16:49:19 +0000 |
| commit | 4dc9fb026cf1834d5b99baee8f265ad6cebf8fc6 (patch) | |
| tree | 6b3445ee14f75ec65f8dd0e9085ad31ea9b76984 | |
| parent | ce9c88348cb1380199b8f57a435a51a7edb470a0 (diff) | |
soc/intel/skylake: reduce postcar stack usage for fsp 2.0
The FSP 2.0 path uses postcar to decompress ramstage. Since postcar
is entirely RAM based there's no need to have an excessively large
stack for the lzma decompression buffer. Therefore, reduce the stack
required to 1 KiB like apollolake.
Change-Id: I45e5c283f8ae87e701c94d6a123463dddde3f221
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/20536
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-by: Lijian Zhao <lijian.zhao@intel.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
| -rw-r--r-- | src/soc/intel/skylake/romstage/romstage_fsp20.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/src/soc/intel/skylake/romstage/romstage_fsp20.c b/src/soc/intel/skylake/romstage/romstage_fsp20.c index 093f7c09b9..1c63250d1f 100644 --- a/src/soc/intel/skylake/romstage/romstage_fsp20.c +++ b/src/soc/intel/skylake/romstage/romstage_fsp20.c @@ -36,12 +36,6 @@ #include <timestamp.h> #include <vboot/vboot_common.h> -/* - * Romstage needs some stack for decompressing ramstage images, since the lzma - * lib keeps its state on the stack during romstage. - */ -#define ROMSTAGE_RAM_STACK_SIZE 0x5000 - #define FSP_SMBIOS_MEMORY_INFO_GUID \ { \ 0xd4, 0x71, 0x20, 0x9b, 0x54, 0xb0, 0x0c, 0x4e, \ @@ -134,7 +128,7 @@ asmlinkage void car_stage_entry(void) pmc_set_disb(); if (!s3wake) save_dimm_info(); - if (postcar_frame_init(&pcf, ROMSTAGE_RAM_STACK_SIZE)) + if (postcar_frame_init(&pcf, 1*KiB)) die("Unable to initialize postcar frame.\n"); /* |