summaryrefslogtreecommitdiff
path: root/src/mainboard/roda/rv11
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2018-05-06 18:13:19 -0500
committerMartin Roth <martinroth@google.com>2018-06-03 14:19:58 +0000
commita50b1f9dd0e6caa3f33ddac4e54c44f881876ede (patch)
tree1a7ba52f879da012b3d631faea799bc74aca55ac /src/mainboard/roda/rv11
parente36a00af719347f1f26d37d4b13fe2d416cdfef1 (diff)
intel bd82x6x/lynxpoint systems: Update ACPI thermal zone handler
Currently the throttle event handler method THRM is defined as an extern on the intel bd82x6x and lynxpoint chipsets, then defined again in the platform with thermal event handling. In newer versions of IASL, this generates an error, as the method is defined in two places. Simply removing the extern causes the call to it to fail on platforms where it isn't actually defined, so add a preprocessor define where it's implemented, and only call the method on those platforms. This also requires moving the thermal handler, which now includes the define to before the gnvs asl file. TEST=Build before and after, make sure correct code is included. Change-Id: I7af4a346496c1352ec20bda8acb338b5d277d99b Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/26123 Reviewed-by: Nico Huber <nico.h@gmx.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/roda/rv11')
-rw-r--r--src/mainboard/roda/rv11/acpi/thermal.asl1
-rw-r--r--src/mainboard/roda/rv11/dsdt.asl5
2 files changed, 4 insertions, 2 deletions
diff --git a/src/mainboard/roda/rv11/acpi/thermal.asl b/src/mainboard/roda/rv11/acpi/thermal.asl
index b87fc4f63d..a1d28e4c53 100644
--- a/src/mainboard/roda/rv11/acpi/thermal.asl
+++ b/src/mainboard/roda/rv11/acpi/thermal.asl
@@ -15,6 +15,7 @@
// Thermal Zone
+#define HAVE_THERMALZONE
Scope (\_TZ)
{
// Convert from Degrees C to 1/10 Kelvin for ACPI
diff --git a/src/mainboard/roda/rv11/dsdt.asl b/src/mainboard/roda/rv11/dsdt.asl
index 4934eed542..ff837959f2 100644
--- a/src/mainboard/roda/rv11/dsdt.asl
+++ b/src/mainboard/roda/rv11/dsdt.asl
@@ -28,11 +28,12 @@ DefinitionBlock(
#include "acpi/platform.asl"
#include "acpi/mainboard.asl"
+ // Thermal handler
+ #include "acpi/thermal.asl"
+
// global NVS and variables
#include <southbridge/intel/bd82x6x/acpi/globalnvs.asl>
- #include "acpi/thermal.asl"
-
#include "acpi/alsd.asl"
#include <cpu/intel/model_206ax/acpi/cpu.asl>