diff options
| author | Martin Roth <martinroth@google.com> | 2016-09-06 10:22:34 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2016-09-07 19:00:05 +0200 |
| commit | a66df49e158472c706a9685c6d4644f193999ec9 (patch) | |
| tree | 9019ee331226623e03b7694b785d6caabe1e57be | |
| parent | b6a77aa7208d22a356e6b9be7868f4f37613aacb (diff) | |
| download | coreboot-a66df49e158472c706a9685c6d4644f193999ec9.tar.xz | |
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>
| -rw-r--r-- | src/arch/x86/acpi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c index 50526a5049..bbeb92f3ba 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 @@ void acpi_create_ivrs(acpi_ivrs_t *ivrs, 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); |