soc/amd/picasso: fix GPE snapshot state

In CB:44488 the cbmem addition was re-filling the object
when it should be memcpy()ing from static object. Correct
that oversight. The side effect from the previous implementation
would be if FSP-M modified the GPE state.

BUG=b:159947207

Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Change-Id: I158a89ae28431896fa9b5789292000fcbf0b066d
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44533
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Aaron Durbin 2020-08-17 17:34:10 -06:00
parent b4e28dfbf8
commit e87ec095a4
1 changed files with 1 additions and 1 deletions

View File

@ -31,7 +31,7 @@ static void add_chipset_state_cbmem(int unused)
state = cbmem_add(CBMEM_ID_POWER_STATE, sizeof(*state));
if (state)
acpi_fill_pm_gpe_state(state);
memcpy(state, &chipset_state, sizeof(*state));
}
ROMSTAGE_CBMEM_INIT_HOOK(add_chipset_state_cbmem);