From f03706a2ba16c9d1c236c2380a41b91a4f417cf3 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 17 Jan 2023 00:37:13 +0100 Subject: [PATCH] soc/amd/picasso/agesa_acpi: use acpi_align_current to align CRAT & IVRS This changes the alignment of the CRAT and IVRS tables from 8 bytes to 16 bytes. TEST=Mandolin still boots Linux and the position and size of the ACPI tables in memory shown by dmesg hasn't changed. Signed-off-by: Felix Held Change-Id: I88df331c8410d8dca41a414543f051f5e4656ff1 Reviewed-on: https://review.coreboot.org/c/coreboot/+/72021 Tested-by: build bot (Jenkins) Reviewed-by: Fred Reitberger --- src/soc/amd/picasso/agesa_acpi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/soc/amd/picasso/agesa_acpi.c b/src/soc/amd/picasso/agesa_acpi.c index fb140e2a0c..d798f104c9 100644 --- a/src/soc/amd/picasso/agesa_acpi.c +++ b/src/soc/amd/picasso/agesa_acpi.c @@ -546,7 +546,7 @@ uintptr_t agesa_write_acpi_tables(const struct device *device, uintptr_t current struct acpi_crat_header *crat; /* CRAT */ - current = ALIGN_UP(current, 8); + current = acpi_align_current(current); crat = (struct acpi_crat_header *)current; acpi_create_crat(crat, acpi_fill_crat); current += crat->header.length; @@ -556,7 +556,7 @@ uintptr_t agesa_write_acpi_tables(const struct device *device, uintptr_t 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;