Frans Hendriks has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32470
Change subject: soc/intel/braswell: Move LPE ACPI code to mainboard ......................................................................
soc/intel/braswell: Move LPE ACPI code to mainboard
The ACPI code for LPE device was included regardless of the availability of the LPE controller. Linux remains requesting the status of device LPEA even if this device is disabled.
Include ACPI LPE controller code at Braswell mainboards with LPE enabled.
BUG=N/A TEST=Linux 4.17+ on Portwell PQ7-M107
Change-Id: Ibec8a8ea437fa275954b0414d6a57a973fe44fc6 Signed-off-by: Frans Hendriks fhendriks@eltan.com --- M src/mainboard/google/cyan/dsdt.asl M src/mainboard/intel/strago/dsdt.asl M src/soc/intel/braswell/acpi/southcluster.asl 3 files changed, 2 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/32470/1
diff --git a/src/mainboard/google/cyan/dsdt.asl b/src/mainboard/google/cyan/dsdt.asl index 397f6d2..3afdaaa 100644 --- a/src/mainboard/google/cyan/dsdt.asl +++ b/src/mainboard/google/cyan/dsdt.asl @@ -42,6 +42,7 @@ #else #include <acpi/dptf/cpu.asl> #endif + #include <acpi/lpe.asl> }
/* Dynamic Platform Thermal Framework */ diff --git a/src/mainboard/intel/strago/dsdt.asl b/src/mainboard/intel/strago/dsdt.asl index c8be3ce..5052ba26 100644 --- a/src/mainboard/intel/strago/dsdt.asl +++ b/src/mainboard/intel/strago/dsdt.asl @@ -15,7 +15,6 @@ * GNU General Public License for more details. */
- #include <arch/acpi.h> DefinitionBlock( "dsdt.aml", @@ -39,6 +38,7 @@ { #include <acpi/southcluster.asl> #include <acpi/dptf/cpu.asl> + #include <acpi/lpe.asl> }
/* Dynamic Platform Thermal Framework */ diff --git a/src/soc/intel/braswell/acpi/southcluster.asl b/src/soc/intel/braswell/acpi/southcluster.asl index f7e3168..9ecf67a 100644 --- a/src/soc/intel/braswell/acpi/southcluster.asl +++ b/src/soc/intel/braswell/acpi/southcluster.asl @@ -286,7 +286,4 @@
/* SCC Devices */ #include "scc.asl" - - /* LPE Device */ - #include "lpe.asl" }