arch/x86/acpi_bert_storage: change return type of bert_errors_present
The return value is a boolean, so use the bool type. Also add the
types.h header to have the bool type defined. Also change type of
bert_region_broken static variable to bool.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I13d6472deeb26ba92d257761df069e32d9b2e5d4
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55023
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
diff --git a/src/arch/x86/acpi_bert_storage.c b/src/arch/x86/acpi_bert_storage.c
index b07b67e..a8b8628 100644
--- a/src/arch/x86/acpi_bert_storage.c
+++ b/src/arch/x86/acpi_bert_storage.c
@@ -8,6 +8,7 @@
#include <acpi/acpi.h>
#include <arch/bert_storage.h>
#include <string.h>
+#include <types.h>
/* BERT region management: Allow the chipset to determine the specific
* location of the BERT region. We find that base and size, then manage
@@ -19,7 +20,7 @@
* resume cycles. If the requirements change, consider using IMD to help
* manage the space.
*/
-static int bert_region_broken;
+static bool bert_region_broken;
static void *bert_region_base;
static size_t bert_region_size;
static size_t bert_region_used;
@@ -32,9 +33,9 @@
return bert_region_broken ? 0 : bert_region_size - bert_region_used;
}
-int bert_errors_present(void)
+bool bert_errors_present(void)
{
- return bert_region_broken ? 0 : !!bert_region_used;
+ return !bert_region_broken && bert_region_used;
}
void bert_errors_region(void **start, size_t *size)
@@ -585,14 +586,14 @@
/* Always start with a blank bert region. Make sure nothing is
* maintained across reboots or resumes.
*/
- bert_region_broken = 0;
+ bert_region_broken = false;
bert_region_used = 0;
bert_reserved_region(&bert_region_base, &bert_region_size);
if (!bert_region_base || !bert_region_size) {
printk(BIOS_ERR, "Bug: Can't find/add BERT storage area\n");
- bert_region_broken = 1;
+ bert_region_broken = true;
return;
}