[coreboot] New patch to review for coreboot: caa65b0 baskingridge: Report static temperature in _TMP
Stefan Reinauer (stefan.reinauer@coreboot.org)
gerrit at coreboot.org
Tue Mar 12 22:51:17 CET 2013
Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2679
-gerrit
commit caa65b0324f83ff4e82cfe34ed9526d2d4886dd3
Author: Duncan Laurie <dlaurie at chromium.org>
Date: Wed Dec 19 09:14:10 2012 -0800
baskingridge: Report static temperature in _TMP
The current code is attempting to convert from an invalid
starting temperature. Since we aren't sure where the temperature
will come from yet just return a static value.
This stops the kernel from going to S5 on boot because it
thinks the temperature is too high.
Change-Id: I433fa407e545458344af5842b353df5bc71bfdad
Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
---
src/mainboard/intel/baskingridge/acpi/thermal.asl | 21 +--------------------
1 file changed, 1 insertion(+), 20 deletions(-)
diff --git a/src/mainboard/intel/baskingridge/acpi/thermal.asl b/src/mainboard/intel/baskingridge/acpi/thermal.asl
index f8e9d97..39dabe3 100644
--- a/src/mainboard/intel/baskingridge/acpi/thermal.asl
+++ b/src/mainboard/intel/baskingridge/acpi/thermal.asl
@@ -63,26 +63,7 @@ Scope (\_TZ)
Method (_TMP, 0, Serialized)
{
- // Get CPU Temperature from PECI via SuperIO TMPIN3
- // FIXME: figure out how to read temp on this board.
- Store (30, Local0)
-
- // Check for invalid readings
- If (LOr (LEqual (Local0, 255), LEqual (Local0, 0))) {
- Return (CTOK (\F2ON))
- }
-
- // PECI raw value is an offset from Tj_max
- Subtract (255, Local0, Local1)
-
- // Handle values greater than Tj_max
- If (LGreaterEqual (Local1, \TMAX)) {
- Return (CTOK (\TMAX))
- }
-
- // Subtract from Tj_max to get temperature
- Subtract (\TMAX, Local1, Local0)
- Return (CTOK (Local0))
+ Return (CTOK (50))
}
Method (_AC0) {
More information about the coreboot
mailing list