diff options
| author | Eran Mitrani <mitrani@google.com> | 2022-11-29 12:54:53 -0800 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-12-02 14:48:37 +0000 |
| commit | 13e151f31ce25dce4116214d7e3329506e029887 (patch) | |
| tree | 3a16064789c8abd372b2f762f93ec10d6d92f769 | |
| parent | 50a32650172dfedc2a6c5335af94fb2d3c7d71e6 (diff) | |
| download | coreboot-13e151f31ce25dce4116214d7e3329506e029887.tar.gz coreboot-13e151f31ce25dce4116214d7e3329506e029887.tar.bz2 coreboot-13e151f31ce25dce4116214d7e3329506e029887.zip | |
soc/intel/alderlake: skip external buses for D-states list
The devices in the list that was introduced in commit c66ea985776
("soc/intel/alderlake: provide a list of D-states to enter
LPM") are all internal. This CL skips the external buses (which caused
the addition of packages to non-existant paths such as
"_SB.PCI0.RP1.MCHC", and warnings from the kernel)
BUG=b:231582182
TEST=Built and tested on anahera by verifying SSDT contents
Change-Id: I3785b2b2af85d96e2e1296b6cfdefcd72080b5fe
Signed-off-by: Eran Mitrani <mitrani@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/70163
Reviewed-by: Tarun Tuli <taruntuli@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
| -rw-r--r-- | src/soc/intel/alderlake/acpi.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/soc/intel/alderlake/acpi.c b/src/soc/intel/alderlake/acpi.c index c874067701d4..d7405ccfde27 100644 --- a/src/soc/intel/alderlake/acpi.c +++ b/src/soc/intel/alderlake/acpi.c @@ -251,6 +251,9 @@ static enum dev_sleep_states get_min_sleep_state(const struct device *dev) return DEFAULT_CPU_D_STATE; case DEVICE_PATH_PCI: + /* skip external buses*/ + if (dev->bus->secondary != 0) + return NONE; for (size_t i = 0; i < ARRAY_SIZE(min_pci_sleep_states); i++) if (min_pci_sleep_states[i].pci_dev == dev->path.pci.devfn) return min_pci_sleep_states[i].min_sleep_state; |