From 9541171de49fbd986d0fa24b4113372bd9ff0081 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Fri, 27 Feb 2026 22:14:08 +0100 Subject: [PATCH] sb/intel/wildcatpoint/acpi: Move platform.asl to mainboards The chipset platform.asl only provided empty _PTS/_WAK stubs and _SWS methods, which mainboards needing custom sleep/wake behavior (e.g. EC methods) cannot use. Only 2 of 5 Wildcat Point boards used it. Move the content to mainboard code and inline the device_nvs and common platform includes in dsdt.asl to align with other Wildcat Point and Lynx Point boards. Keeping device NVS in mainboard code also simplifies future Lynx/Wildcat unification. Tested with BUILD_TIMELESS=1, Purism Librem 15 v2 remains identical. Change-Id: I753302a13567efb3b7903364be8cef486d2b76e5 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/91469 Reviewed-by: Matt DeVillier Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel --- .../google/auron}/acpi/platform.asl | 3 -- src/mainboard/google/auron/dsdt.asl | 4 ++- .../purism/librem_bdw/acpi/platform.asl | 35 +++++++++++++++++++ src/mainboard/purism/librem_bdw/dsdt.asl | 4 ++- 4 files changed, 41 insertions(+), 5 deletions(-) rename src/{southbridge/intel/wildcatpoint => mainboard/google/auron}/acpi/platform.asl (81%) create mode 100644 src/mainboard/purism/librem_bdw/acpi/platform.asl diff --git a/src/southbridge/intel/wildcatpoint/acpi/platform.asl b/src/mainboard/google/auron/acpi/platform.asl similarity index 81% rename from src/southbridge/intel/wildcatpoint/acpi/platform.asl rename to src/mainboard/google/auron/acpi/platform.asl index af60f5cee7..a0a94ef3cc 100644 --- a/src/southbridge/intel/wildcatpoint/acpi/platform.asl +++ b/src/mainboard/google/auron/acpi/platform.asl @@ -1,8 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0-only */ -#include -#include - /* * The _PTS method (Prepare To Sleep) is called before the OS is * entering a sleep state. The sleep state number is passed in Arg0 diff --git a/src/mainboard/google/auron/dsdt.asl b/src/mainboard/google/auron/dsdt.asl index eebeeeacee..97de5cd2e6 100644 --- a/src/mainboard/google/auron/dsdt.asl +++ b/src/mainboard/google/auron/dsdt.asl @@ -11,7 +11,9 @@ DefinitionBlock( ) { #include - #include + #include + #include + #include "acpi/platform.asl" // Thermal handler #include "acpi/thermal.asl" diff --git a/src/mainboard/purism/librem_bdw/acpi/platform.asl b/src/mainboard/purism/librem_bdw/acpi/platform.asl new file mode 100644 index 0000000000..a0a94ef3cc --- /dev/null +++ b/src/mainboard/purism/librem_bdw/acpi/platform.asl @@ -0,0 +1,35 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +/* + * The _PTS method (Prepare To Sleep) is called before the OS is + * entering a sleep state. The sleep state number is passed in Arg0 + */ + +Method (_PTS, 1) +{ +} + +/* The _WAK method is called on system wakeup */ + +Method (_WAK, 1) +{ + Return (Package (){ 0, 0 }) +} + +Scope (\_SB) +{ + Method (_SWS) + { + /* Index into PM1 for device that caused wake */ + Return (\PM1I) + } +} + +Scope (\_GPE) +{ + Method (_SWS) + { + /* Index into GPE for device that caused wake */ + Return (\GPEI) + } +} diff --git a/src/mainboard/purism/librem_bdw/dsdt.asl b/src/mainboard/purism/librem_bdw/dsdt.asl index 18719a8f1e..23d3df1280 100644 --- a/src/mainboard/purism/librem_bdw/dsdt.asl +++ b/src/mainboard/purism/librem_bdw/dsdt.asl @@ -11,7 +11,9 @@ DefinitionBlock( ) { #include - #include + #include + #include + #include "acpi/platform.asl" #include #include