From efd2720e4790924a434823d701664de8ee2eb9e4 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Wed, 13 Jul 2022 07:20:47 +0200 Subject: arch/x86: Mark prepare_and_run_postcar noreturn This moves the die() statement to a common place. Change-Id: I24c9f00bfee169b4ca57b469c089188ec62ddada Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/65812 Tested-by: build bot (Jenkins) Reviewed-by: Felix Held Reviewed-by: Julius Werner --- src/arch/x86/include/arch/romstage.h | 2 +- src/arch/x86/postcar_loader.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) (limited to 'src/arch/x86') diff --git a/src/arch/x86/include/arch/romstage.h b/src/arch/x86/include/arch/romstage.h index d637d19b8a..063d4edd6b 100644 --- a/src/arch/x86/include/arch/romstage.h +++ b/src/arch/x86/include/arch/romstage.h @@ -43,7 +43,7 @@ void fill_postcar_frame(struct postcar_frame *pcf); * 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(void); +void __noreturn prepare_and_run_postcar(void); /* * Systems without a native coreboot cache-as-ram teardown may implement diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c index af8152d72b..c6a128bb69 100644 --- a/src/arch/x86/postcar_loader.c +++ b/src/arch/x86/postcar_loader.c @@ -63,7 +63,7 @@ static void run_postcar_phase(struct postcar_frame *pcf); /* 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(void) +void __noreturn prepare_and_run_postcar(void) { struct postcar_frame pcf; @@ -76,6 +76,7 @@ void prepare_and_run_postcar(void) run_postcar_phase(&pcf); /* We do not return here. */ + die("Failed to load postcar\n!"); } static void finalize_load(uintptr_t *reloc_params, uintptr_t mtrr_frame_ptr) -- cgit v1.2.3