diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2016-06-28 07:38:03 +0300 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2016-06-29 07:33:58 +0200 |
commit | c30bfcaa9ebbe4d3e2ea0cac11784e13e56fe281 (patch) | |
tree | 2df596520c5a22288a005d115a83dcc89b3aaf9e | |
parent | 1891bfdac65f02769a5581d274b668a9e0dbf176 (diff) | |
download | coreboot-c30bfcaa9ebbe4d3e2ea0cac11784e13e56fe281.tar.gz coreboot-c30bfcaa9ebbe4d3e2ea0cac11784e13e56fe281.tar.bz2 coreboot-c30bfcaa9ebbe4d3e2ea0cac11784e13e56fe281.zip |
AMD k8 fam10: Refactor S3 recovery
Change-Id: I09c218ca05391e8d80880be0aa5bdfd5079acf85
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/15465
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r-- | src/cpu/amd/car/post_cache_as_ram.c | 36 |
1 files changed, 16 insertions, 20 deletions
diff --git a/src/cpu/amd/car/post_cache_as_ram.c b/src/cpu/amd/car/post_cache_as_ram.c index 1880ccb4410c..15b264280123 100644 --- a/src/cpu/amd/car/post_cache_as_ram.c +++ b/src/cpu/amd/car/post_cache_as_ram.c @@ -63,27 +63,32 @@ static int memcmp_(void *d, const void *s, size_t len) return memcmp(d, s, len); } -static void prepare_romstage_ramstack(void *resume_backup_memory) +static void prepare_romstage_ramstack(int s3resume) { size_t backup_top = backup_size(); print_car_debug("Prepare CAR migration and stack regions..."); - if (resume_backup_memory) { - memcpy_(resume_backup_memory + HIGH_MEMORY_SAVE - backup_top, - (void *)(CONFIG_RAMTOP - backup_top), backup_top); + if (s3resume) { + void *resume_backup_memory = cbmem_find(CBMEM_ID_RESUME); + if (resume_backup_memory) + memcpy_(resume_backup_memory + HIGH_MEMORY_SAVE - backup_top, + (void *)(CONFIG_RAMTOP - backup_top), backup_top); } memset_((void *)(CONFIG_RAMTOP - backup_top), 0, backup_top); print_car_debug(" Done\n"); } -static void prepare_ramstage_region(void *resume_backup_memory) +static void prepare_ramstage_region(int s3resume) { size_t backup_top = backup_size(); print_car_debug("Prepare ramstage memory region..."); - if (resume_backup_memory) { - memcpy_(resume_backup_memory, (void *) CONFIG_RAMBASE, HIGH_MEMORY_SAVE - backup_top); + if (s3resume) { + void *resume_backup_memory = cbmem_find(CBMEM_ID_RESUME); + if (resume_backup_memory) + memcpy_(resume_backup_memory, (void *) CONFIG_RAMBASE, + HIGH_MEMORY_SAVE - backup_top); memset_((void*) CONFIG_RAMBASE, 0, HIGH_MEMORY_SAVE - backup_top); } else { memset_((void*)0, 0, CONFIG_RAMTOP - backup_top); @@ -115,7 +120,6 @@ static void vErrata343(void) void post_cache_as_ram(void) { - void *resume_backup_memory = NULL; uint32_t family = amd_fam1x_cpu_family(); /* Verify that the BSP didn't overrun the lower stack @@ -134,11 +138,10 @@ void post_cache_as_ram(void) printk(BIOS_DEBUG, "Romstage handoff structure not added!\n"); int s3resume = acpi_is_wakeup_s3(); - if (s3resume) { + if (s3resume) cbmem_recovery(s3resume); - resume_backup_memory = cbmem_find(CBMEM_ID_RESUME); - } - prepare_romstage_ramstack(resume_backup_memory); + + prepare_romstage_ramstack(s3resume); /* from here don't store more data in CAR */ if (family >= 0x1f && family <= 0x3f) { @@ -168,9 +171,6 @@ void post_cache_as_ram(void) void cache_as_ram_new_stack (void) { - void *resume_backup_memory = NULL; - - print_car_debug("Top about %08x ... Done\n", (uint32_t) &resume_backup_memory); print_car_debug("Disabling cache as ram now\n"); disable_cache_as_ram_bsp(); @@ -179,11 +179,7 @@ void cache_as_ram_new_stack (void) set_var_mtrr(0, 0x00000000, CACHE_TMP_RAMTOP, MTRR_TYPE_WRBACK); enable_cache(); - if (acpi_is_wakeup_s3()) { - resume_backup_memory = cbmem_find(CBMEM_ID_RESUME); - print_car_debug("Resume backup memory location: %p\n", resume_backup_memory); - } - prepare_ramstage_region(resume_backup_memory); + prepare_ramstage_region(acpi_is_wakeup_s3()); set_sysinfo_in_ram(1); // So other core0 could start to train mem |