coreboot: move TS_END_ROMSTAGE to one spot
While the romstage code flow is not consistent across all mainboards/chipsets there is only one way of running ramstage from romstage -- run_ramstage(). Move the timestamp_add_now(TS_END_ROMSTAGE) to be within run_ramstage(). BUG=chrome-os-partner:44827 BRANCH=None TEST=Built and booted glados. TS_END_ROMSTAGE still present in timestamp table. Change-Id: I4b584e274ce2107e83ca6425491fdc71a138e82c Signed-off-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/11700 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
parent
a40032780f
commit
9796f60c62
|
@ -274,7 +274,6 @@ void romstage_common(const struct romstage_params *params)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(wake_from_s3);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
||||
static inline void prepare_for_resume(struct romstage_handoff *handoff)
|
||||
|
|
|
@ -96,6 +96,8 @@ void run_ramstage(void)
|
|||
struct prog ramstage =
|
||||
PROG_INIT(ASSET_RAMSTAGE, CONFIG_CBFS_PREFIX "/ramstage");
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
/* Only x86 systems currently take the same firmware path on resume. */
|
||||
if (IS_ENABLED(CONFIG_ARCH_X86) && IS_ENABLED(CONFIG_EARLY_CBMEM_INIT))
|
||||
run_ramstage_from_resume(romstage_handoff_find_or_add(),
|
||||
|
|
|
@ -226,8 +226,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
rs780_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -225,8 +225,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb800_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -224,8 +224,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -333,8 +333,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
// die("After MCT init before CAR disabled.");
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -224,8 +224,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -344,7 +344,4 @@ void main(unsigned long bist)
|
|||
|
||||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization(s3resume);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
}
|
||||
|
|
|
@ -368,8 +368,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
/* Restore default SuperIO access */
|
||||
outb(0xaa, port);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
}
|
||||
|
|
|
@ -226,8 +226,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -226,8 +226,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -223,8 +223,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
rs780_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -227,8 +227,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
rs780_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -221,8 +221,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -221,8 +221,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -224,8 +224,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -53,7 +53,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -179,7 +179,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -247,6 +247,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -198,5 +198,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -278,7 +278,5 @@ void main(void)
|
|||
|
||||
simple_spi_test();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -252,5 +252,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -111,7 +111,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -102,7 +102,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -106,7 +106,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -102,7 +102,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -112,7 +112,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -102,7 +102,5 @@ void main(void)
|
|||
|
||||
cbmem_initialize_empty();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
run_ramstage();
|
||||
}
|
||||
|
|
|
@ -218,8 +218,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
bcm5785_early_setup();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram();
|
||||
}
|
||||
|
||||
|
|
|
@ -224,8 +224,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -306,8 +306,6 @@ void romstage_main_continue(EFI_STATUS status, VOID *HobListPtr) {
|
|||
*(u32*)cbmem_hob_ptr = (u32)HobListPtr;
|
||||
post_code(0x4f);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
/* Load the ramstage. */
|
||||
copy_and_run();
|
||||
while (1);
|
||||
|
|
|
@ -256,5 +256,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -229,8 +229,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
rs780_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -243,5 +243,4 @@ void main(unsigned long bist)
|
|||
northbridge_romstage_finalize(boot_mode==2);
|
||||
|
||||
post_code(0x3f);
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -284,6 +284,4 @@ void main(unsigned long bist)
|
|||
|
||||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization(s3resume);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -310,6 +310,4 @@ void main(unsigned long bist)
|
|||
#if CONFIG_LPC_TPM
|
||||
init_tpm(s3resume);
|
||||
#endif
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -284,7 +284,4 @@ void main(unsigned long bist)
|
|||
|
||||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization(s3resume);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
}
|
||||
|
|
|
@ -243,8 +243,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
amdmct_cbmem_store_info(sysinfo);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
}
|
||||
|
|
|
@ -297,6 +297,4 @@ void main(unsigned long bist)
|
|||
quick_ram_check();
|
||||
}
|
||||
#endif
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -274,5 +274,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -284,5 +284,4 @@ void main(unsigned long bist)
|
|||
if (CONFIG_LPC_TPM) {
|
||||
init_tpm(boot_mode == 2);
|
||||
}
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -240,8 +240,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
amdmct_cbmem_store_info(sysinfo);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy + execute stage 2
|
||||
post_code(0x42); // Should never see this post code.
|
||||
}
|
||||
|
|
|
@ -304,8 +304,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
amdmct_cbmem_store_info(sysinfo);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram(); /* BSP switch stack to ram, copy then execute CB. */
|
||||
post_code(0x42); /* Should never see this post code. */
|
||||
}
|
||||
|
|
|
@ -239,8 +239,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
sr5650_before_pci_init();
|
||||
sb7xx_51xx_before_pci_init();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x42);
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
|
|
|
@ -239,8 +239,6 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
|
|||
|
||||
amdmct_cbmem_store_info(sysinfo);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_cache_as_ram(); // BSP switch stack to ram, copy then execute LB.
|
||||
post_code(0x43); // Should never see this post code.
|
||||
}
|
||||
|
|
|
@ -96,7 +96,6 @@ void main(unsigned long bist)
|
|||
#if CONFIG_EARLY_CBMEM_INIT
|
||||
cbmem_recovery(0);
|
||||
#endif
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
/* FIXME: See if this is needed or take this out please */
|
||||
/* Disable Memcard and SDIO */
|
||||
pci_mod_config8(LPC, 0x51, 0, (1 << 7) | (1 << 4));
|
||||
|
|
|
@ -135,5 +135,4 @@ void main(unsigned long bist)
|
|||
#endif
|
||||
|
||||
post_code(0x3f);
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
}
|
||||
|
|
|
@ -250,8 +250,6 @@ void romstage_common(struct romstage_params *params)
|
|||
|
||||
void asmlinkage romstage_after_car(void)
|
||||
{
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
/* Load the ramstage. */
|
||||
copy_and_run();
|
||||
while (1);
|
||||
|
|
|
@ -132,8 +132,6 @@ void romstage_common(struct romstage_params *params)
|
|||
|
||||
void asmlinkage romstage_after_car(void)
|
||||
{
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
/* Load the ramstage. */
|
||||
copy_and_run();
|
||||
while (1);
|
||||
|
|
|
@ -207,8 +207,6 @@ asmlinkage void romstage_after_car(void *chipset_context)
|
|||
soc_after_temp_ram_exit();
|
||||
soc_display_mtrrs();
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
/* Load the ramstage. */
|
||||
copy_and_run();
|
||||
die("ERROR - Failed to load ramstage!");
|
||||
|
|
|
@ -267,8 +267,6 @@ void romstage_main_continue(EFI_STATUS status, void *hob_list_ptr) {
|
|||
else
|
||||
printk(BIOS_DEBUG, "Romstage handoff structure not added!\n");
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x4f);
|
||||
|
||||
/* Load the ramstage. */
|
||||
|
|
|
@ -130,8 +130,6 @@ void romstage_main_continue(EFI_STATUS status, void *hob_list_ptr) {
|
|||
*(u32*)cbmem_hob_ptr = (u32)hob_list_ptr;
|
||||
post_code(0x4e);
|
||||
|
||||
timestamp_add_now(TS_END_ROMSTAGE);
|
||||
|
||||
post_code(0x4f);
|
||||
|
||||
/* Load the ramstage. */
|
||||
|
|
Loading…
Reference in New Issue