Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63415 )
Change subject: arch/x86/acpi_bert_storage.c: Use BOOT_STATE over CBMEM hooks ......................................................................
arch/x86/acpi_bert_storage.c: Use BOOT_STATE over CBMEM hooks
With the purpose of linking ramstage inside the bootblock we likely want to skip some ramstage CBMEM hooks and keep those only for recovering data from earlier stages.
Change-Id: I317173d468073906d76228d1c8cc7bc28aae9e75 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/arch/x86/acpi_bert_storage.c 1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/63415/1
diff --git a/src/arch/x86/acpi_bert_storage.c b/src/arch/x86/acpi_bert_storage.c index 8559c06..7b6b63b 100644 --- a/src/arch/x86/acpi_bert_storage.c +++ b/src/arch/x86/acpi_bert_storage.c @@ -1,5 +1,6 @@ /* SPDX-License-Identifier: GPL-2.0-only */
+#include <bootstate.h> #include <cbmem.h> #include <console/console.h> #include <cpu/x86/name.h> @@ -577,7 +578,7 @@ *size = 0; }
-static void bert_storage_setup(int unused) +static void bert_storage_setup(void *unused) { /* Always start with a blank bert region. Make sure nothing is * maintained across reboots or resumes. @@ -596,4 +597,4 @@ memset(bert_region_base, 0, bert_region_size); }
-RAMSTAGE_CBMEM_INIT_HOOK(bert_storage_setup) +BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_EXIT, bert_storage_setup, NULL);