[coreboot-gerrit] Change in coreboot[master]: soc/intel/braswell/acpi/dptf/thermal.asl: Thermal event

[Frans Hendriks (Code Review)]

Frans Hendriks has uploaded this change for review. ( https://review.coreboot.org/27415


Change subject: soc/intel/braswell/acpi/dptf/thermal.asl: Thermal event
......................................................................

soc/intel/braswell/acpi/dptf/thermal.asl: Thermal event

The thermal event support can not be disabled.
Define and depend code are placed in same file.

Move HAVE_THERM_EVENT_HANDLER to mainboard file.

Change-Id: Icb532e5bc7fd171ee2921f9a4b9b2150ba9f05c5
Signed-off-by: Frans Hendriks <fhendriks at eltan.com>
---
M src/mainboard/google/cyan/acpi/dptf.asl
M src/soc/intel/braswell/acpi/dptf/thermal.asl
2 files changed, 7 insertions(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/27415/1

diff --git a/src/mainboard/google/cyan/acpi/dptf.asl b/src/mainboard/google/cyan/acpi/dptf.asl
index 3ecb040..dd6bb68 100644
--- a/src/mainboard/google/cyan/acpi/dptf.asl
+++ b/src/mainboard/google/cyan/acpi/dptf.asl
@@ -3,6 +3,7 @@
  *
  * Copyright (C) 2012 Google Inc.
  * Copyright (C) 2015 Intel Corp.
+ * Copyright (C) 2018 Eltan B.V.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -14,6 +15,8 @@
  * GNU General Public License for more details.
  */
 
+#define HAVE_THERM_EVENT_HANDLER
+
 /* Include Variant DPTF */
 #include <variant/acpi/dptf.asl>
 
diff --git a/src/soc/intel/braswell/acpi/dptf/thermal.asl b/src/soc/intel/braswell/acpi/dptf/thermal.asl
index e104756..ec17411 100644
--- a/src/soc/intel/braswell/acpi/dptf/thermal.asl
+++ b/src/soc/intel/braswell/acpi/dptf/thermal.asl
@@ -1,6 +1,8 @@
 /*
  * This file is part of the coreboot project.
  *
+ * Copyright (C) 2018 Eltan B.V.
+ 
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License as
  * published by the Free Software Foundation; version 2 of
@@ -13,7 +15,7 @@
  */
 
 /* Thermal Threshold Event Handler */
-#define HAVE_THERM_EVENT_HANDLER
+#ifdef HAVE_THERM_EVENT_HANDLER
 Method (TEVT, 1, NotSerialized)
 {
 	Store (ToInteger (Arg0), Local0)
@@ -34,6 +36,7 @@
 	}
 #endif
 }
+#endif
 
 /* Thermal device initialization - Disable Aux Trip Points */
 Method (TINI)

-- 
To view, visit https://review.coreboot.org/27415
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icb532e5bc7fd171ee2921f9a4b9b2150ba9f05c5
Gerrit-Change-Number: 27415
Gerrit-PatchSet: 1
Gerrit-Owner: Frans Hendriks <fhendriks at eltan.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180709/e390a79d/attachment.html>