From 07dda337b78be64c6064fcce14ccb5d1f7fdace2 Mon Sep 17 00:00:00 2001 From: Benjamin Doron Date: Fri, 5 Feb 2021 00:23:46 +0000 Subject: [PATCH] acpi: Fix BERT size_t printf format error Fix compilation on GCC 10.2.1 and address the underlying issue. The printf format specifier for a size_t type is z. Change-Id: Ieb1db6c0c3eb4947bd3617e418bac238b70ec08f Signed-off-by: Benjamin Doron Reviewed-on: https://review.coreboot.org/c/coreboot/+/50051 Tested-by: build bot (Jenkins) Reviewed-by: HAOUAS Elyes Reviewed-by: Angel Pons --- src/arch/x86/acpi_bert_storage.c | 2 +- src/soc/intel/common/block/systemagent/memmap.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/arch/x86/acpi_bert_storage.c b/src/arch/x86/acpi_bert_storage.c index ec31917444..f1dcf47c23 100644 --- a/src/arch/x86/acpi_bert_storage.c +++ b/src/arch/x86/acpi_bert_storage.c @@ -200,7 +200,7 @@ void *new_cper_fw_error_crashlog(acpi_generic_error_status_t *status, size_t cl_ { void *cl_data = bert_allocate_storage(cl_size); if (!cl_data) { - printk(BIOS_ERR, "Error: Crashlog entry (size %lu) would exceed available region\n", + printk(BIOS_ERR, "Error: Crashlog entry (size %zu) would exceed available region\n", cl_size); return NULL; } diff --git a/src/soc/intel/common/block/systemagent/memmap.c b/src/soc/intel/common/block/systemagent/memmap.c index ce842e850f..a67d050fcc 100644 --- a/src/soc/intel/common/block/systemagent/memmap.c +++ b/src/soc/intel/common/block/systemagent/memmap.c @@ -63,7 +63,7 @@ void bert_reserved_region(void **start, size_t *size) *start = cbmem_add(CBMEM_ID_ACPI_BERT, BERT_REGION_MAX_SIZE); *size = BERT_REGION_MAX_SIZE; - printk(BIOS_DEBUG, "Reserving BERT start %lx, size %lx\n", (uintptr_t)*start, *size); + printk(BIOS_DEBUG, "Reserving BERT start %lx, size %zx\n", (uintptr_t)*start, *size); } void fill_postcar_frame(struct postcar_frame *pcf)