[coreboot-gerrit] Change in ...coreboot[master]: ACPI: Enum only used ACPI tables

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31592 )

Change subject: ACPI: Enum only used ACPI tables
......................................................................

ACPI: Enum only used ACPI tables

enum contained redundant names and wasn't exhaustive anyway.

Change-Id: I4d74ff61c555c5953932efbd7edccfd3157cb5be
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/31592
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
---
M src/arch/x86/include/arch/acpi.h
1 file changed, 5 insertions(+), 9 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved

diff --git a/src/arch/x86/include/arch/acpi.h b/src/arch/x86/include/arch/acpi.h
index a061a27..cc0f1a9 100644
--- a/src/arch/x86/include/arch/acpi.h
+++ b/src/arch/x86/include/arch/acpi.h
@@ -77,16 +77,12 @@
    COREBOOT_ACPI_ID_MAX		= 0xFFFF, /* BOOTFFFF */
 };

-/* Table 5-30 DESCRIPTION_HEADER Signatures for tables defined by ACPI 6.2a
- *  Additional tables mssing in 5-30:  MADT, RSDP, VFCT, NHLT
- */
 enum acpi_tables {
-	APIC, BERT, BGRT, CPEP, DSDT, ECDT, EINJ, ERST, FACP, FADT, FACS,
-	FPDT, GTDT, HEST, MSCT, MPST, NFIT, OEMX, PCCT, PMTT, PSDT, RASF,
-	RSDT, SBST, SDEV, SLIT, SRAT, SSDT, XSDT, BOOT, CSRT, DBG2, DBGP,
-	DMAR, DPPT, DRTM, ETDT, HPET, IBFT, IORT, IVRS, LPIT, MCFG, MCHI,
-	MSDM, SDEI, SLIC, SPCR, SPMI, STAO, TCPA, TPM2, WAET, WDAT, WDRT,
-	WPBT, WSMT, XENV, MADT, RSDP, VFCT, NHLT
+	/* Tables defined by ACPI and used by coreboot */
+	BERT, DBG2, DMAR, DSDT, FACS, FADT, HEST, HPET, IVRS, MADT, MCFG,
+	RSDP, RSDT, SLIT, SRAT, SSDT, TCPA, TPM2, XSDT, ECDT,
+	/* Additional proprietary tables used by coreboot */
+	VFCT, NHLT
 };

/* RSDP (Root System Description Pointer) */