From 2b9653cf34070d85a46ed6a9a03e3a6da1ba49e2 Mon Sep 17 00:00:00 2001 From: Maximilian Brune Date: Wed, 27 Aug 2025 20:32:19 +0200 Subject: [PATCH] arch/x86/acpi_bert_storage.c: rename check -> proc_err_info The name check is a bit confusing, since it is not a check structure. The check structure is below it. Change-Id: I000e9e5f2ce8210fce76ef81b4242150d02fceed Signed-off-by: Maximilian Brune Reviewed-on: https://review.coreboot.org/c/coreboot/+/90478 Tested-by: build bot (Jenkins) Reviewed-by: Felix Held --- src/arch/x86/acpi_bert_storage.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/arch/x86/acpi_bert_storage.c b/src/arch/x86/acpi_bert_storage.c index b271cffc66..e9b9fc563a 100644 --- a/src/arch/x86/acpi_bert_storage.c +++ b/src/arch/x86/acpi_bert_storage.c @@ -386,7 +386,7 @@ cper_ia32x64_proc_error_info_t *new_cper_ia32x64_check( cper_ia32x64_proc_error_section_t *x86err, enum cper_x86_check_type type) { - cper_ia32x64_proc_error_info_t *check; + cper_ia32x64_proc_error_info_t *proc_err_info; static const char * const check_names[] = { "cache", "TLB", @@ -409,19 +409,19 @@ cper_ia32x64_proc_error_info_t *new_cper_ia32x64_check( return NULL; } - check = bert_allocate_storage(sizeof(*check)); - if (!check) { + proc_err_info = bert_allocate_storage(sizeof(*proc_err_info)); + if (!proc_err_info) { printk(BIOS_ERR, "New IA32X64 %s check entry would exceed available region\n", check_names[type]); return NULL; } - revise_error_sizes(status, sizeof(*check)); + revise_error_sizes(status, sizeof(*proc_err_info)); - guidcpy(&check->type, &check_guids[type]); + guidcpy(&proc_err_info->type, &check_guids[type]); cper_bump_ia32x64_chk_count(x86err); - return check; + return proc_err_info; } /* Helper to append an ACPI Generic Error Data Entry plus a CPER IA32/X64