diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2021-05-27 20:37:21 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-06-08 18:22:24 +0000 |
commit | 62eb0ed93e53997925967318e20e13fa0d08799a (patch) | |
tree | 0d2c73ca444e8745d0a355c05b68a6f1e9905180 /src/arch/x86 | |
parent | fba479267b0294f3d6cf43bf751ffd3dc300cf25 (diff) |
arch/x86/include/bert_storage: introduce bert_should_generate_acpi_table
Since bert_errors_present() is only available when ACPI_BERT is selected
the ACPI table generation code needs to check that before calling the
function, so add bert_should_generate_acpi_table that returns false when
ACPI_BERT isn't selected or the return value of bert_errors_present()
when ACPI_BERT is selected.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ia955f627c190ea38e05b5aaedc7cb2d030274e83
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55024
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/arch/x86')
-rw-r--r-- | src/arch/x86/include/arch/bert_storage.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/arch/x86/include/arch/bert_storage.h b/src/arch/x86/include/arch/bert_storage.h index 63783331e4..ea4ec3d2ba 100644 --- a/src/arch/x86/include/arch/bert_storage.h +++ b/src/arch/x86/include/arch/bert_storage.h @@ -56,6 +56,11 @@ void bert_errors_region(void **start, size_t *size); size_t bert_storage_remaining(void); /* Find if errors were added, a BERT region is present, and ACPI table needed */ bool bert_errors_present(void); +/* The BERT table should only be generated when BERT support is enabled and there's an error */ +static inline bool bert_should_generate_acpi_table(void) +{ + return CONFIG(ACPI_BERT) && bert_errors_present(); +} /* Get the number of entries associated with status */ static inline size_t bert_entry_count(acpi_generic_error_status_t *status) |