From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web09.1743.1656440421667725313 for ; Tue, 28 Jun 2022 11:20:21 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pranav.madhu@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7D970153B; Tue, 28 Jun 2022 11:20:21 -0700 (PDT) Received: from usa.arm.com (unknown [10.162.16.33]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id A64AF3F792; Tue, 28 Jun 2022 11:20:19 -0700 (PDT) From: "Pranav Madhu" To: devel@edk2.groups.io Cc: Ard Biesheuvel , Sami Mujawar , Pierre Gondois , Thomas Abraham Subject: [edk2-platforms][PATCH V2 1/7] Platform/Sgi: Fix ACPI Low Power Idle states for SGI575 Date: Tue, 28 Jun 2022 23:50:06 +0530 Message-Id: <20220628182012.1179982-2-pranav.madhu@arm.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220628182012.1179982-1-pranav.madhu@arm.com> References: <20220628182012.1179982-1-pranav.madhu@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable SGI-575 platform supports only the platform co-ordinated LPI. So fix the LPI Level ID value accordingly. Additionally, as this platform does not support residency counter, clear the residency counter frequency from _LPI object's control method. Signed-off-by: Pranav Madhu Reviewed-by: Pierre Gondois Reviewed-by: Thomas Abraham --- Platform/ARM/SgiPkg/AcpiTables/Sgi575/Dsdt.asl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Platform/ARM/SgiPkg/AcpiTables/Sgi575/Dsdt.asl b/Platform/AR= M/SgiPkg/AcpiTables/Sgi575/Dsdt.asl index 80075ee1238b..f4e84e806a65 100644 --- a/Platform/ARM/SgiPkg/AcpiTables/Sgi575/Dsdt.asl +++ b/Platform/ARM/SgiPkg/AcpiTables/Sgi575/Dsdt.asl @@ -57,7 +57,7 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 2, "ARMLTD", "= ARMSGI", EFI_ACPI_ARM_OEM 1150, // Wake latency (uS) 1, // Flags 1, // Arch Context Flags - 100, // Residency Counter Frequency + 0, // Residency Counter Frequency 0, // No Parent State 0x00000020, // Integer Entry method ResourceTemplate () { // Null Residency Counter @@ -72,14 +72,14 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 2, "ARMLTD",= "ARMSGI", EFI_ACPI_ARM_OEM =20 Name (PLPI, Package () { /* LPI for Processor, support 2 LPI states= */ 0, // Version - 1, // Level Index + 0, // Level Index 2, // Count Package () { // WFI for CPU 1, // Min residency (uS) 1, // Wake latency (uS) 1, // Flags 0, // Arch Context lost Flags (no loss) - 100, // Residency Counter Frequency + 0, // Residency Counter Frequency 0, // No parent state ResourceTemplate () { // Register Entry method Register (FFixedHW, @@ -102,7 +102,7 @@ DefinitionBlock("DsdtTable.aml", "DSDT", 2, "ARMLTD",= "ARMSGI", EFI_ACPI_ARM_OEM 350, // Wake latency (uS) 1, // Flags 1, // Arch Context lost Flags (Core context l= ost) - 100, // Residency Counter Frequency + 0, // Residency Counter Frequency 1, // Parent node can be in any shallower sta= te ResourceTemplate () { // Register Entry method Register (FFixedHW, --=20 2.25.1