diff options
author | Patrick Georgi <pgeorgi@google.com> | 2017-10-25 18:47:27 -0400 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2017-10-26 04:57:49 +0000 |
commit | c6202e8c00ba32a7a0b8516992a3a2f72caa7df2 (patch) | |
tree | 9ddeec12b13410873c4bb3691566e76846fe9539 /src/soc | |
parent | 435299461f4e6c061898866435b48cd9883d5f74 (diff) |
soc/intel/apollolake: avoid double accounting for power state
intel/common's pmclib already keeps track of the power state (since
commit f073872e22728fe8ade85022740af95cc129e9a5 and doing it twice can
mess up the data that ends up in cbmem (and from there, everything else),
so don't.
BUG=b:67976359
BRANCH=none
TEST=builds
Change-Id: I69c804a2a3bee43add940d8c827b7250f2fe9024
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/22179
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/soc')
-rw-r--r-- | src/soc/intel/apollolake/romstage.c | 22 |
1 files changed, 2 insertions, 20 deletions
diff --git a/src/soc/intel/apollolake/romstage.c b/src/soc/intel/apollolake/romstage.c index cc02ff5db0..f315e61096 100644 --- a/src/soc/intel/apollolake/romstage.c +++ b/src/soc/intel/apollolake/romstage.c @@ -53,8 +53,6 @@ #include <compiler.h> #include "chip.h" -static struct chipset_power_state power_state CAR_GLOBAL; - static const uint8_t hob_variable_guid[16] = { 0x7d, 0x14, 0x34, 0xa0, 0x0c, 0x69, 0x54, 0x41, 0x8d, 0xe6, 0xc0, 0x44, 0x64, 0x1d, 0xe9, 0x42, @@ -111,22 +109,6 @@ static void disable_watchdog(void) outl(reg, ACPI_BASE_ADDRESS + TCO1_CNT); } -static void migrate_power_state(int is_recovery) -{ - struct chipset_power_state *ps_cbmem; - struct chipset_power_state *ps_car; - - ps_car = car_get_var_ptr(&power_state); - ps_cbmem = cbmem_add(CBMEM_ID_POWER_STATE, sizeof(*ps_cbmem)); - - if (ps_cbmem == NULL) { - printk(BIOS_DEBUG, "Unable to add power state to cbmem!\n"); - return; - } - memcpy(ps_cbmem, ps_car, sizeof(*ps_cbmem)); -} -ROMSTAGE_CBMEM_INIT_HOOK(migrate_power_state); - /* * Punit Initialization code. This all isn't documented, but * this is the recipe. @@ -211,7 +193,7 @@ asmlinkage void car_stage_entry(void) struct postcar_frame pcf; uintptr_t top_of_ram; bool s3wake; - struct chipset_power_state *ps = car_get_var_ptr(&power_state); + struct chipset_power_state *ps = pmc_get_power_state(); void *smm_base; size_t smm_size, var_size; const void *new_var_data; @@ -312,7 +294,7 @@ static void check_full_retrain(const FSPM_UPD *mupd) if (mupd->FspmArchUpd.BootMode != FSP_BOOT_WITH_FULL_CONFIGURATION) return; - ps = car_get_var_ptr(&power_state); + ps = pmc_get_power_state(); if (ps->gen_pmcon1 & WARM_RESET_STS) { printk(BIOS_INFO, "Full retrain unsupported on warm reboot.\n"); |