From ba00d10c41d1df1ca8a5eadc95a40430fbd059c5 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Tue, 15 Feb 2022 11:04:15 +0100 Subject: arch/x86/postcar_loader.c: Reduce the scope of functions Some functions are only called locally. Change-Id: I96a4e40a225536f62abb2a15c55d333b8604e8cc Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/61963 Tested-by: build bot (Jenkins) Reviewed-by: Raul Rangel --- src/arch/x86/include/arch/romstage.h | 17 ----------------- src/arch/x86/postcar_loader.c | 6 ++++-- 2 files changed, 4 insertions(+), 19 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 7a2a41773b52..5951766a3c83 100644 --- a/src/arch/x86/include/arch/romstage.h +++ b/src/arch/x86/include/arch/romstage.h @@ -20,12 +20,6 @@ struct postcar_frame { struct var_mtrr_context *mtrr; }; -/* - * Initialize postcar_frame object. - * Returns 0 on success, < 0 on error. - */ -int postcar_frame_init(struct postcar_frame *pcf); - /* * Add variable MTRR covering the provided range with MTRR type. */ @@ -51,17 +45,6 @@ void fill_postcar_frame(struct postcar_frame *pcf); */ void prepare_and_run_postcar(struct postcar_frame *pcf); -/* - * Load and run a program that takes control of execution that - * tears down CAR and loads ramstage. The postcar_frame object - * indicates how to set up the frame. If caching is enabled at - * the time of the call it is up to the platform code to handle - * coherency with dirty lines in the cache using some mechanism - * such as platform_prog_run() because run_postcar_phase() - * utilizes prog_run() internally. - */ -void run_postcar_phase(struct postcar_frame *pcf); - /* * Systems without a native coreboot cache-as-ram teardown may implement * this to use an alternate method. diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c index d46bf3c4953a..b5bfe3e6c960 100644 --- a/src/arch/x86/postcar_loader.c +++ b/src/arch/x86/postcar_loader.c @@ -19,7 +19,7 @@ static size_t var_mtrr_ctx_size(void) return sizeof(struct var_mtrr_context) + mtrr_count * 2 * sizeof(msr_t); } -int postcar_frame_init(struct postcar_frame *pcf) +static int postcar_frame_init(struct postcar_frame *pcf) { struct var_mtrr_context *ctx; @@ -57,6 +57,8 @@ static void postcar_frame_common_mtrrs(struct postcar_frame *pcf) postcar_frame_add_romcache(pcf, MTRR_TYPE_WRPROT); } +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(struct postcar_frame *pcf) @@ -129,7 +131,7 @@ static void postcar_cache_invalid(void) board_reset(); } -void run_postcar_phase(struct postcar_frame *pcf) +static void run_postcar_phase(struct postcar_frame *pcf) { struct prog prog = PROG_INIT(PROG_POSTCAR, CONFIG_CBFS_PREFIX "/postcar"); -- cgit v1.2.3