sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

arch/x86/acpi_bert_storage.c: Use BOOT_STATE over CBMEM hooks

Browse Source 
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>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63415
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
This commit is contained in:
Arthur Heymans
2022-04-06 20:40:35 +02:00
committed by Felix Held
parent 993b43f2be
commit f4543e7426
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/arch/x86/acpi_bert_storage.c
View File

@ -1,5 +1,6 @@
/* SPDX-License-Identifier: GPL-2.0-only */ /* SPDX-License-Identifier: GPL-2.0-only */
#include <bootstate.h>
#include <cbmem.h> #include <cbmem.h>
#include <console/console.h> #include <console/console.h>
#include <cpu/x86/name.h> #include <cpu/x86/name.h>
@ -577,7 +578,7 @@ __weak void bert_reserved_region(void **start, size_t *size)
*size = 0; *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 /* Always start with a blank bert region. Make sure nothing is
* maintained across reboots or resumes. * maintained across reboots or resumes.
@ -596,4 +597,4 @@ static void bert_storage_setup(int unused)
memset(bert_region_base, 0, bert_region_size); 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);