From 5212ece6cf27fa91ffd07d97d4fbe171129cf1fa Mon Sep 17 00:00:00 2001 From: Tim Wawrzynczak Date: Thu, 16 Jul 2020 11:54:04 -0600 Subject: dptf: Fix scope of TCPU device In the initial DPTF refactor, the scope of the TCPU device was incorrectly set as \_SB, instead of \_SB.PCI0. However, because of the way that the acpi_inject_dsdt() callback currently works (it injects contents before the dsdt.aml file), the Scope where the TCPU device lives (\_SB.PCI0) doesn't exist yet. Therefore, to avoid playing games with *when* things are defined in the DSDT, switch to defining all of the DPTF devices in the SSDT. Signed-off-by: Tim Wawrzynczak Change-Id: Ia4922b4dc6544d79d44d39e6ad18c6ab9fee0fd7 Reviewed-on: https://review.coreboot.org/c/coreboot/+/43529 Reviewed-by: Duncan Laurie Reviewed-by: Karthik Ramasubramanian Tested-by: build bot (Jenkins) --- src/include/acpi/acpigen_dptf.h | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src/include') diff --git a/src/include/acpi/acpigen_dptf.h b/src/include/acpi/acpigen_dptf.h index 89b64f3728..1790df77e8 100644 --- a/src/include/acpi/acpigen_dptf.h +++ b/src/include/acpi/acpigen_dptf.h @@ -9,6 +9,10 @@ /* A common idiom is to use a default value if none is provided (i.e., == 0) */ #define DEFAULT_IF_0(thing, default_) ((thing) ? (thing) : (default_)) +/* Hardcoded paths */ +#define DPTF_DEVICE_PATH "\\_SB.DPTF" +#define TCPU_SCOPE "\\_SB.PCI0" + /* List of available participants (i.e., they can participate in policies) */ enum dptf_participant { DPTF_NONE, -- cgit v1.2.3