From d4330e7dfb121db43399dd18855b2b9af0f02c4a Mon Sep 17 00:00:00 2001 From: Pratikkumar Prajapati Date: Tue, 30 May 2023 10:38:23 -0700 Subject: soc/intel/common/crashlog: Fix checking PMC record size Check pmc_record_size variable for collecting PMC records, instead of cpu_record_size variable. Bug=None TEST=Able to build. With Meteor Lake SOC related patch, able to capture and decode crashlog. Change-Id: I4c35ba2bcf757231aa2872802eb82d4d50742cd9 Signed-off-by: Pratikkumar Prajapati Reviewed-on: https://review.coreboot.org/c/coreboot/+/75526 Reviewed-by: Subrata Banik Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/acpi/acpi_bert.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/soc') diff --git a/src/soc/intel/common/block/acpi/acpi_bert.c b/src/soc/intel/common/block/acpi/acpi_bert.c index 7743ccc72fa0..16df60a9d247 100644 --- a/src/soc/intel/common/block/acpi/acpi_bert.c +++ b/src/soc/intel/common/block/acpi/acpi_bert.c @@ -68,8 +68,8 @@ static enum cb_err record_crashlog_into_bert(void **region, size_t *length) pmc_record_size = cl_get_pmc_record_size(); if (pmc_record_size) { - /* Allocate new FW ERR structure in case CPU crashlog is present */ - if (cpu_record_size && !bert_append_fw_err(status)) { + /* Allocate new FW ERR structure in case PMC crashlog is present */ + if (pmc_record_size && !bert_append_fw_err(status)) { printk(BIOS_ERR, "Crashlog PMC entry would " "exceed available region\n"); return CB_ERR; -- cgit v1.2.3