diff options
| author | Joel Kitching <kitching@google.com> | 2018-08-17 15:15:02 +0800 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-08-22 15:32:30 +0000 |
| commit | 5846d5727a05e395d13317daba049e0e56e15d33 (patch) | |
| tree | 51f9e3027b8708891dabd9351bd45092de3f3581 /src/arch/x86/acpi_s3.c | |
| parent | 8f560d9b9c20c7e72b031e60cf0e828d7d27ec8e (diff) | |
acpi: remove CBMEM_ID_ACPI_GNVS_PTR entry
Since we can retrieve the address of ACPI GNVS directly
from CBMEM_ID_ACPI_GNVS, there is no need to store and
update a pointer separately.
TEST=Compile and run on Eve
Signed-off-by: Joel Kitching <kitching@google.com>
Change-Id: I59f3d0547a4a724e66617c791ad82c9f504cadea
Reviewed-on: https://review.coreboot.org/28189
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/arch/x86/acpi_s3.c')
| -rw-r--r-- | src/arch/x86/acpi_s3.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/arch/x86/acpi_s3.c b/src/arch/x86/acpi_s3.c index f6ed1089cf..ebf291cdce 100644 --- a/src/arch/x86/acpi_s3.c +++ b/src/arch/x86/acpi_s3.c @@ -226,13 +226,13 @@ void __weak mainboard_suspend_resume(void) void acpi_resume(void *wake_vec) { if (IS_ENABLED(CONFIG_HAVE_SMI_HANDLER)) { - u32 *gnvs_address = cbmem_find(CBMEM_ID_ACPI_GNVS_PTR); + void *gnvs_address = cbmem_find(CBMEM_ID_ACPI_GNVS); /* Restore GNVS pointer in SMM if found */ - if (gnvs_address && *gnvs_address) { - printk(BIOS_DEBUG, "Restore GNVS pointer to 0x%08x\n", - *gnvs_address); - smm_setup_structures((void *)*gnvs_address, NULL, NULL); + if (gnvs_address) { + printk(BIOS_DEBUG, "Restore GNVS pointer to %p\n", + gnvs_address); + smm_setup_structures(gnvs_address, NULL, NULL); } } |