[coreboot-gerrit] Change in coreboot[master]: ec/mec1308: Fix fan control ACPI
Matt DeVillier (Code Review)
gerrit at coreboot.org
Fri Jul 7 00:20:54 CEST 2017
Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/20482
Change subject: ec/mec1308: Fix fan control ACPI
......................................................................
ec/mec1308: Fix fan control ACPI
Returing FSL# for _STA causes Windows to BSOD. Re-work _STA to instead
return 0/1 based on FLVL, using google/beltino as a model.
Also correct serialization type for _CRS.
Change-Id: Ibf3af15bab3590f7c1c4401e1978dbcf2a495216
Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
---
M src/ec/smsc/mec1308/acpi/ec.asl
1 file changed, 36 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/20482/1
diff --git a/src/ec/smsc/mec1308/acpi/ec.asl b/src/ec/smsc/mec1308/acpi/ec.asl
index a8de9fa..697e4b1 100644
--- a/src/ec/smsc/mec1308/acpi/ec.asl
+++ b/src/ec/smsc/mec1308/acpi/ec.asl
@@ -58,7 +58,7 @@
FCOS, 1, // Fan Speed OS Control
}
- Method (_CRS, 0, NotSerialized)
+ Method (_CRS, 0, Serialized)
{
Name (ECMD, ResourceTemplate()
{
@@ -87,7 +87,13 @@
PowerResource (FNP0, 0, 0)
{
- Method (_STA) { Return (FSL0) }
+ Method (_STA) {
+ If (LLessEqual (\FLVL, 0)) {
+ Return (One)
+ } Else {
+ Return (Zero)
+ }
+ }
Method (_ON) {
If (FCOS) {
Store (One, FSL0)
@@ -106,7 +112,13 @@
PowerResource (FNP1, 0, 0)
{
- Method (_STA) { Return (FSL1) }
+ Method (_STA) {
+ If (LLessEqual (\FLVL, 0)) {
+ Return (One)
+ } Else {
+ Return (Zero)
+ }
+ }
Method (_ON) {
If (FCOS) {
Store (One, FSL1)
@@ -125,7 +137,13 @@
PowerResource (FNP2, 0, 0)
{
- Method (_STA) { Return (FSL2) }
+ Method (_STA) {
+ If (LLessEqual (\FLVL, 0)) {
+ Return (One)
+ } Else {
+ Return (Zero)
+ }
+ }
Method (_ON) {
If (FCOS) {
Store (One, FSL2)
@@ -144,7 +162,13 @@
PowerResource (FNP3, 0, 0)
{
- Method (_STA) { Return (FSL3) }
+ Method (_STA) {
+ If (LLessEqual (\FLVL, 0)) {
+ Return (One)
+ } Else {
+ Return (Zero)
+ }
+ }
Method (_ON) {
If (FCOS) {
Store (One, FSL3)
@@ -163,7 +187,13 @@
PowerResource (FNP4, 0, 0)
{
- Method (_STA) { Return (FSL4) }
+ Method (_STA) {
+ If (LLessEqual (\FLVL, 0)) {
+ Return (One)
+ } Else {
+ Return (Zero)
+ }
+ }
Method (_ON) {
If (FCOS) {
Store (One, FSL4)
--
To view, visit https://review.coreboot.org/20482
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibf3af15bab3590f7c1c4401e1978dbcf2a495216
Gerrit-Change-Number: 20482
Gerrit-PatchSet: 1
Gerrit-Owner: Matt DeVillier <matt.devillier at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170706/8261284e/attachment.html>
More information about the coreboot-gerrit
mailing list