Attention is currently required from: Eran Mitrani, Subrata Banik.
Tarun Tuli has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71644 )
Change subject: soc/intel/meteorlake: provide a list of D-states to enter LPM ......................................................................
Patch Set 7:
(1 comment)
File src/soc/intel/meteorlake/acpi.c:
https://review.coreboot.org/c/coreboot/+/71644/comment/30a73e0b_5c0515bb PS1, Line 172: // { SA_DEVFN_CPU_PCIE1_0, 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 }, : // { SA_DEVFN_CPU_PCIE6_0, ACPI_DEVICE_SLEEP_D3 }, : // { SA_DEVFN_CPU_PCIE6_2, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TBT0, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TBT1, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TBT2, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TBT3, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_GNA, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TCSS_XHCI, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_TCSS_XDCI, ACPI_DEVICE_SLEEP_D3 }, : { SA_DEVFN_TCSS_DMA0, ACPI_DEVICE_SLEEP_D3 }, : { SA_DEVFN_TCSS_DMA1, ACPI_DEVICE_SLEEP_D3 }, : { PCI_DEVFN_VMD, ACPI_DEVICE_SLEEP_D3 }, : // { PCH_DEVFN_I2C6, ACPI_DEVICE_SLEEP_D3 }, : // { PCH_DEVFN_I2C7, ACPI_DEVICE_SLEEP_D3 },
Tarun, […]
They were based on some educated assumptions. These would be most impacted by the downstream peripheral. I think they would be fine to leave as D3.