From 6b1ceacb9b30ec873b341811645413ae3e9e0331 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sun, 6 May 2018 17:58:36 -0500 Subject: [PATCH] chromeec platforms: Update ACPI throttle handler call Currently the throttle event handler method THRT is defined as an extern, 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. Change-Id: I6337c52edaf9350843848b31c5d87bbfca403930 Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/26121 Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh --- src/ec/google/chromeec/acpi/ec.asl | 13 ++++++------- src/mainboard/google/slippy/acpi/ec.asl | 3 +++ 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/src/ec/google/chromeec/acpi/ec.asl b/src/ec/google/chromeec/acpi/ec.asl index dc0f60903e..6589ccf1a8 100644 --- a/src/ec/google/chromeec/acpi/ec.asl +++ b/src/ec/google/chromeec/acpi/ec.asl @@ -20,7 +20,6 @@ */ // Mainboard specific throttle handler -External (\_TZ.THRT, MethodObj) #ifdef DPTF_ENABLE_CHARGER External (\_SB.DPTF.TCHG, DeviceObj) #endif @@ -325,19 +324,19 @@ Device (EC0) // Throttle Start Method (_Q12, 0, NotSerialized) { +#ifdef EC_ENABLE_THROTTLING_HANDLER Store ("EC: THROTTLE START", Debug) - If (CondRefOf (\_TZ.THRT)) { - \_TZ.THRT (1) - } + \_TZ.THRT (1) +#endif } // Throttle Stop Method (_Q13, 0, NotSerialized) { +#ifdef EC_ENABLE_THROTTLING_HANDLER Store ("EC: THROTTLE STOP", Debug) - If (CondRefOf (\_TZ.THRT)) { - \_TZ.THRT (0) - } + \_TZ.THRT (0) +#endif } #ifdef EC_ENABLE_PD_MCU_DEVICE diff --git a/src/mainboard/google/slippy/acpi/ec.asl b/src/mainboard/google/slippy/acpi/ec.asl index d4f62c4c91..7189ef1a9e 100644 --- a/src/mainboard/google/slippy/acpi/ec.asl +++ b/src/mainboard/google/slippy/acpi/ec.asl @@ -24,5 +24,8 @@ */ #define EC_ENABLE_WAKE_PIN 0x69 +/* Slippy variants have a throttle handler in thermal.asl */ +#define EC_ENABLE_THROTTLING_HANDLER + /* ACPI code for EC functions */ #include