Paul Fagerburg has submitted this change. ( https://review.coreboot.org/c/coreboot/+/72603 )
Change subject: soc/intel/mtl: remove DPTF from D-states list used to enter LPM ......................................................................
soc/intel/mtl: remove DPTF from D-states list used to enter LPM
The D-state list lists the devices with the corresponding D-state that the devices should be in, in order to enter LPM. DPTF is not mentioned in Intel's document 595644 as one of the devices. This CL removes it to avoid a potential error seen in ADL devices as mentioned in commit 3fd5b0c4cdeb ("soc/intel/adl: remove DPTF from D-states list used to enter LPM")
TEST=Built and tested on Rex, saw SSDT generated properly. BUG=b:231582182
Signed-off-by: Eran Mitrani mitrani@google.com Change-Id: I9192ed9a7fb59ebba14f6d5082b400534b16ca72 Reviewed-on: https://review.coreboot.org/c/coreboot/+/72603 Reviewed-by: Subrata Banik subratabanik@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/intel/meteorlake/acpi.c 1 file changed, 24 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Subrata Banik: Looks good to me, approved
diff --git a/src/soc/intel/meteorlake/acpi.c b/src/soc/intel/meteorlake/acpi.c index 14aece3..c679af3 100644 --- a/src/soc/intel/meteorlake/acpi.c +++ b/src/soc/intel/meteorlake/acpi.c @@ -170,7 +170,6 @@ static struct min_sleep_state min_pci_sleep_states[] = { { SA_DEVFN_ROOT, ACPI_DEVICE_SLEEP_D3 }, { SA_DEVFN_IGD, ACPI_DEVICE_SLEEP_D3 }, - { PCI_DEVFN_DPTF, ACPI_DEVICE_SLEEP_D3 }, { PCI_DEVFN_IPU, ACPI_DEVICE_SLEEP_D3 }, { PCI_DEVFN_TBT0, ACPI_DEVICE_SLEEP_D3 }, { PCI_DEVFN_TBT1, ACPI_DEVICE_SLEEP_D3 },