Reviewed-by: Evan Lloyd <[email protected]> > -----Original Message----- > From: AlexeiFedorov <[email protected]> > Sent: 04 July 2018 14:05 > To: [email protected] > Cc: Arvind Chauhan <[email protected]>; Thomas Abraham > <[email protected]>; [email protected]; > [email protected]; Matteo Carlini <[email protected]>; > Stephanie Hughes-Fitt <[email protected]>; nd > <[email protected]>; Thomas Abraham <[email protected]>; Evan > Lloyd <[email protected]>; Sami Mujawar <[email protected]> > Subject: [PATCH] Platform/ARM: Correct LevelID in PLPI packages of DSDT > > From: Alexei Fedorov <[email protected]> > > Juno's DSDT contains 2 PLPI packages in Clusters #0 and #1 and _OSC method > reports support for platform coordinated mode only. > According to the description of LevelID field in ACPI 6.2 Errata A > Specification > #8.4.4.3, "In a platform that only supports platform coordinated mode, this > field must be 0." > > This patch fixes the above issue by changing value of LevelID fields from 1 to > 0. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Alexei Fedorov <[email protected]> > --- > All the changes can be reviewed at: > https://github.com/AlexeiFedorov/edk2- > platforms/tree/282_correct_levelid_v1 > > Notes: > v1: > - Change LevelID Value of PLPI package from 1 to 0. > > Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl > b/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl > index > 07e32bae21f891461fde0183028e4c0f817e45a7..702b057757457fee40ddfc10 > e91d38c5dd7ca0b8 100644 > --- a/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl > +++ b/Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl > @@ -1,7 +1,7 @@ > /** @file > Differentiated System Description Table Fields (DSDT) > > - Copyright (c) 2014-2015, ARM Ltd. All rights reserved.<BR> > + Copyright (c) 2014-2018, ARM Ltd. All rights reserved.<BR> > This program and the accompanying materials > are licensed and made available under the terms and conditions of the BSD > License > which accompanies this distribution. The full text of the license may be > found at @@ -65,7 +65,7 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 1, > "ARMLTD", "ARM-JUNO", EFI_ACPI_ARM_O > }) > Name(PLPI, Package() { > 0, // Version > - 1, // Level Index > + 0, // Level Index > 2, // Count > Package() { // WFI for CPU > 1, // Min residency (uS) > @@ -157,7 +157,7 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 1, > "ARMLTD", "ARM-JUNO", EFI_ACPI_ARM_O > }) > Name(PLPI, Package() { > 0, // Version > - 1, // Level Index > + 0, // Level Index > 2, // Count > Package() { // WFI for CPU > 1, // Min residency (uS) > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' >
_______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
