commit a66df49e158472c706a9685c6d4644f193999ec9
author Martin Roth <martinroth@google.com> Tue Sep 06 10:22:34 2016 -0600
committer Martin Roth <martinroth@google.com> Wed Sep 07 19:00:05 2016 +0200
tree 9019ee331226623e03b7694b785d6caabe1e57be
parent b6a77aa7208d22a356e6b9be7868f4f37613aacb

x86/acpi.c: use #define for IVRS revision field

The revision field was correct, but the comment was wrong. The revision
1 means that the IVRS table only uses fixed length device entries.
Update the field to use the IVRS revision #define.

Change-Id: I4c030b31e3e3f0a402dac36ab69f43d99e131c22
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/16507
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>

src/arch/x86/acpi.c

diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c
index 50526a5..bbeb92f 100644
--- a/src/arch/x86/acpi.c
+++ b/src/arch/x86/acpi.c
@@ -39,6 +39,7 @@
 #include <console/console.h>
 #include <string.h>
 #include <arch/acpi.h>
+#include <arch/acpi_ivrs.h>
 #include <arch/acpigen.h>
 #include <device/pci.h>
 #include <cbmem.h>
@@ -603,7 +604,7 @@
 	memcpy(header->asl_compiler_id, ASLC, 4);
 
 	header->length = sizeof(acpi_ivrs_t);
-	header->revision = 1; /* ACPI 1.0: N/A, ACPI 2.0/3.0/4.0: 1 */
+	header->revision = IVRS_FORMAT_FIXED;
 
 	current = acpi_fill_ivrs(ivrs, current);