GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

google/panther: Be safe about invalid thermal readings 

In case we get an invalid thermal reading, let's run the fan
at full speed rather than at low speed. This might impact the
user experiance slightly in cases where the bad reading does
not happen while the system is hot, but it will increase stability
in the cases where the system is actually overheating.

Also, set the critical temperature below tjmax, because otherwise
thermal shutdown by the OS will never be triggered.

Signed-off-by: Stefan Reinauer <reinauer@google.com>

Change-Id: Iab262f1f17a5dff875c596d9e8d50e4e50ee90f9
Reviewed-on: https://chromium-review.googlesource.com/188556
Reviewed-by: Shawn Nematbakhsh <shawnn@chromium.org>
Commit-Queue: Stefan Reinauer <reinauer@chromium.org>
Tested-by: Stefan Reinauer <reinauer@chromium.org>
(cherry picked from commit 721fc2361ea9c6fea75409be57726294ce840f03)
Signed-off-by: Isaac Christensen <isaac.christensen@se-eng.com>
Reviewed-on: http://review.coreboot.org/6962
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
Stefan Reinauer 2014-03-03 15:08:10 -08:00 committed by Isaac Christensen
parent 3f1500f54f
commit be1d1e7f83
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/mainboard/google/panther/acpi/thermal.asl
View File

@ -76,12 +76,12 @@ Scope (\_TZ)
// Check for "no reading available" // Check for "no reading available"
If (LEqual (Local0, 0x80)) { If (LEqual (Local0, 0x80)) {
Return (CTOK (\F2ON)) Return (CTOK (\F0ON))
} }
// Check for invalid readings // Check for invalid readings
If (LOr (LEqual (Local0, 255), LEqual (Local0, 0))) { If (LOr (LEqual (Local0, 255), LEqual (Local0, 0))) {
Return (CTOK (\F2ON)) Return (CTOK (\F0ON))
} }
// PECI raw value is an offset from Tj_max // PECI raw value is an offset from Tj_max