arch/x86/postcar_loader.c: Change prepare_and_run_postcar signature
The postcar frame can now be a local variable to that function.
Change-Id: I873298970fff76b9ee1cae7da156613eb557ffbc
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61964
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c
index b5bfe3e..af8152d 100644
--- a/src/arch/x86/postcar_loader.c
+++ b/src/arch/x86/postcar_loader.c
@@ -21,6 +21,8 @@
static int postcar_frame_init(struct postcar_frame *pcf)
{
+ memset(pcf, 0, sizeof(*pcf));
+
struct var_mtrr_context *ctx;
ctx = cbmem_add(CBMEM_ID_ROMSTAGE_RAM_STACK, var_mtrr_ctx_size());
@@ -61,16 +63,18 @@
/* prepare_and_run_postcar() determines the stack to use after
* cache-as-ram is torn down as well as the MTRR settings to use. */
-void prepare_and_run_postcar(struct postcar_frame *pcf)
+void prepare_and_run_postcar(void)
{
- if (postcar_frame_init(pcf))
+ struct postcar_frame pcf;
+
+ if (postcar_frame_init(&pcf))
die("Unable to initialize postcar frame.\n");
- fill_postcar_frame(pcf);
+ fill_postcar_frame(&pcf);
- postcar_frame_common_mtrrs(pcf);
+ postcar_frame_common_mtrrs(&pcf);
- run_postcar_phase(pcf);
+ run_postcar_phase(&pcf);
/* We do not return here. */
}