| Message ID | 1500658062-20929-1-git-send-email-jhugo@codeaurora.org |
|---|---|
| State | Accepted |
| Headers | show |
On 2017-07-21 10:27 AM, Jeffrey Hugo wrote: > ACPI 6.2 has the same MADT support as 6.1, but the test needs to be updated > to recognize 6.2 as valid, which allows the MADT test to validate a > platform advertising 6.2 via FADT. > > Signed-off-by: Jeffrey Hugo <jhugo@codeaurora.org> > --- > src/acpi/madt/madt.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c > index a24fa00..d9b6db1 100644 > --- a/src/acpi/madt/madt.c > +++ b/src/acpi/madt/madt.c > @@ -117,7 +117,7 @@ > */ > > #define FADT_MAX_MAJOR_REVISION ((uint8_t)6) > -#define FADT_MAX_MINOR_REVISION ((uint8_t)1) > +#define FADT_MAX_MINOR_REVISION ((uint8_t)2) > #define MADT_MAX_REVISION ((uint8_t)4) > > #define SUBTABLE_UNDEFINED 0x00 > @@ -212,6 +212,14 @@ static struct acpi_madt_subtable_lengths spec_info[] = { > .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, > 16, 16, 12, 80, 24, 24, 16, 20 } > }, > + { /* for ACPI 6.2 */ > + .major_version = 6, > + .minor_version = 2, > + .madt_version = 4, > + .num_types = 16, > + .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, > + 16, 16, 12, 80, 24, 24, 16, 20 } > + }, > { /* terminator */ > .major_version = 0, > .minor_version = 0, > Acked-by: Alex Hung <alex.hung@canonical.com>
On 07/22/2017 01:27 AM, Jeffrey Hugo wrote: > ACPI 6.2 has the same MADT support as 6.1, but the test needs to be updated > to recognize 6.2 as valid, which allows the MADT test to validate a > platform advertising 6.2 via FADT. > > Signed-off-by: Jeffrey Hugo <jhugo@codeaurora.org> > --- > src/acpi/madt/madt.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c > index a24fa00..d9b6db1 100644 > --- a/src/acpi/madt/madt.c > +++ b/src/acpi/madt/madt.c > @@ -117,7 +117,7 @@ > */ > > #define FADT_MAX_MAJOR_REVISION ((uint8_t)6) > -#define FADT_MAX_MINOR_REVISION ((uint8_t)1) > +#define FADT_MAX_MINOR_REVISION ((uint8_t)2) > #define MADT_MAX_REVISION ((uint8_t)4) > > #define SUBTABLE_UNDEFINED 0x00 > @@ -212,6 +212,14 @@ static struct acpi_madt_subtable_lengths spec_info[] = { > .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, > 16, 16, 12, 80, 24, 24, 16, 20 } > }, > + { /* for ACPI 6.2 */ > + .major_version = 6, > + .minor_version = 2, > + .madt_version = 4, > + .num_types = 16, > + .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, > + 16, 16, 12, 80, 24, 24, 16, 20 } > + }, > { /* terminator */ > .major_version = 0, > .minor_version = 0, > Acked-by: Ivan Hu <ivan.hu@canonical.com>
diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c index a24fa00..d9b6db1 100644 --- a/src/acpi/madt/madt.c +++ b/src/acpi/madt/madt.c @@ -117,7 +117,7 @@ */ #define FADT_MAX_MAJOR_REVISION ((uint8_t)6) -#define FADT_MAX_MINOR_REVISION ((uint8_t)1) +#define FADT_MAX_MINOR_REVISION ((uint8_t)2) #define MADT_MAX_REVISION ((uint8_t)4) #define SUBTABLE_UNDEFINED 0x00 @@ -212,6 +212,14 @@ static struct acpi_madt_subtable_lengths spec_info[] = { .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, 16, 16, 12, 80, 24, 24, 16, 20 } }, + { /* for ACPI 6.2 */ + .major_version = 6, + .minor_version = 2, + .madt_version = 4, + .num_types = 16, + .lengths = { 8, 12, 10, 8, 6, 12, 16, SUBTABLE_VARIABLE, + 16, 16, 12, 80, 24, 24, 16, 20 } + }, { /* terminator */ .major_version = 0, .minor_version = 0,
ACPI 6.2 has the same MADT support as 6.1, but the test needs to be updated to recognize 6.2 as valid, which allows the MADT test to validate a platform advertising 6.2 via FADT. Signed-off-by: Jeffrey Hugo <jhugo@codeaurora.org> --- src/acpi/madt/madt.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-)