arch/x86/postcar: unexpose postcar_commit_mtrrs() symbol

postcar_commit_mtrrs() isn't used outside the postcar_loader
compilation unit. Make it static to reduce API surface area.
Additionally the return value was not being utilized. Modify
the return type to be void to align with the current usage.

Change-Id: If07f34467941d00de731489867e485cfff80ea63
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35130
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
This commit is contained in:
Aaron Durbin 2019-08-27 20:22:40 -06:00
parent 1225afe482
commit df2bfb9a1a
2 changed files with 1 additions and 8 deletions

View File

@ -57,12 +57,6 @@ void postcar_frame_add_romcache(struct postcar_frame *pcf, int type);
*/
void postcar_frame_common_mtrrs(struct postcar_frame *pcf);
/*
* Push used MTRR and Max MTRRs on to the stack
* and return pointer to stack top.
*/
void *postcar_commit_mtrrs(struct postcar_frame *pcf);
/*
* fill_postcar_frame() is called after raminit completes and right before
* calling run_postcar_phase(). Implementation should call postcar_frame_add_mtrr()

View File

@ -147,7 +147,7 @@ void prepare_and_run_postcar(struct postcar_frame *pcf)
/* We do not return here. */
}
void *postcar_commit_mtrrs(struct postcar_frame *pcf)
static void postcar_commit_mtrrs(struct postcar_frame *pcf)
{
/*
* Place the number of used variable MTRRs on stack then max number
@ -155,7 +155,6 @@ void *postcar_commit_mtrrs(struct postcar_frame *pcf)
*/
stack_push(pcf, pcf->num_var_mtrrs);
stack_push(pcf, pcf->max_var_mtrrs);
return (void *) pcf->stack;
}
static void finalize_load(uintptr_t *stack_top_ptr, uintptr_t stack_top)