From 192945e61cef3f90b0a7464b98801b4bd0150ff7 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 17 Jan 2023 02:50:06 +0100 Subject: [PATCH] soc/amd/phoenix/agesa_acpi: use acpi_align_current to align IVRS & ALIB This changes the alignment of the IVRS table from 8 bytes to 16 bytes and aligns the ALIB table to a 16 byte boundary. Signed-off-by: Felix Held Change-Id: I766260aefcac6876609d6b45202b41a3e9e44385 Reviewed-on: https://review.coreboot.org/c/coreboot/+/72025 Tested-by: build bot (Jenkins) Reviewed-by: Fred Reitberger --- src/soc/amd/phoenix/agesa_acpi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/soc/amd/phoenix/agesa_acpi.c b/src/soc/amd/phoenix/agesa_acpi.c index 8838d9dc49..3c58a20f58 100644 --- a/src/soc/amd/phoenix/agesa_acpi.c +++ b/src/soc/amd/phoenix/agesa_acpi.c @@ -16,10 +16,11 @@ uintptr_t agesa_write_acpi_tables(const struct device *device, uintptr_t current acpi_ivrs_t *ivrs; /* add ALIB SSDT from HOB */ + current = acpi_align_current(current); current = add_agesa_fsp_acpi_table(AMD_FSP_ACPI_ALIB_HOB_GUID, "ALIB", rsdp, current); /* IVRS */ - current = ALIGN_UP(current, 8); + current = acpi_align_current(current); ivrs = (acpi_ivrs_t *)current; acpi_create_ivrs(ivrs, acpi_fill_ivrs); current += ivrs->header.length;