[coreboot-gerrit] [XS] Change in coreboot[master]: acpi/acpi.c: Fix printing all ACPI tables

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/75860?usp=email )

Change subject: acpi/acpi.c: Fix printing all ACPI tables
......................................................................

acpi/acpi.c: Fix printing all ACPI tables

Loop over tables in xsdt instead of maintaining a list of local
variables to loop over. Some tables were not generated directly in the
write_acpi_tables function, like IVRS or SRAT. Now those tables are
printed too and the code is simpler.

Signed-off-by: Arthur Heymans 
Change-Id: Ie0a6e2b6e2b72b5c8f59e730bea9b51007b507b6
---
M src/acpi/acpi.c
1 file changed, 2 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/75860/1

diff --git a/src/acpi/acpi.c b/src/acpi/acpi.c
index aa1cd1c..78135d7 100644
--- a/src/acpi/acpi.c
+++ b/src/acpi/acpi.c
@@ -2136,12 +2136,8 @@

if (CONFIG(DEBUG_ACPICA_COMPATIBLE)) {
    	printk(BIOS_DEBUG, "Printing ACPI tables in ACPICA compatible format\n");
-		void *acpi_tables[] = {	rsdt, xsdt, fadt, facs, slic, ssdt, dsdt,
-					mcfg, tcpa, tpm2, madt, lpit, bert };
-		for (size_t i = 0; i < ARRAY_SIZE(acpi_tables); i++) {
-			if (acpi_tables[i] == NULL)
-				continue;
-			acpidump_print(acpi_tables[i]);
+		for (size_t i = 0; xsdt->entry[i] != 0; i++) {
+			acpidump_print((void *)(uintptr_t)xsdt->entry[i]);
    	}
    	printk(BIOS_DEBUG, "Done printing ACPI tables in ACPICA compatible format\n");
    }