summaryrefslogtreecommitdiff
path: root/src/southbridge/intel/lynxpoint/acpi
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2015-12-08 15:26:46 -0700
committerMartin Roth <martinroth@google.com>2015-12-10 16:31:14 +0100
commit5a98bf2c0a9c1616e7fc807e0b5d8a2cbcf9062e (patch)
tree39bc19a45db5d6355e42f1824b9441d024d98b5b /src/southbridge/intel/lynxpoint/acpi
parent10f7f5044ec8364419a2722b2822db3d9d94a7ed (diff)
downloadcoreboot-5a98bf2c0a9c1616e7fc807e0b5d8a2cbcf9062e.tar.xz
ACPI: Fix IASL Warning about unused method for _TZ checks
According to the ACPI Spec for CondRefOf, the result argument is optional. In all of these locations, it was getting set but not used, creating a warning in new versions of IASL. Since it's an optional argument, just remove it. dsdt.aml 640: If (CondRefOf (^GBUF, Local0)) { Warning 3144 - Method Local is set but never used ^ (Local0) Change-Id: Iddf46a4faab19019882847917397eee0614302b9 Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/12695 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/southbridge/intel/lynxpoint/acpi')
-rw-r--r--src/southbridge/intel/lynxpoint/acpi/globalnvs.asl4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/southbridge/intel/lynxpoint/acpi/globalnvs.asl b/src/southbridge/intel/lynxpoint/acpi/globalnvs.asl
index cbc3bd7e7f..468c755b16 100644
--- a/src/southbridge/intel/lynxpoint/acpi/globalnvs.asl
+++ b/src/southbridge/intel/lynxpoint/acpi/globalnvs.asl
@@ -222,12 +222,12 @@ External (\_TZ.SKIN)
Method (TZUP)
{
/* Update Primary Thermal Zone */
- If (CondRefOf (\_TZ.THRM, Local0)) {
+ If (CondRefOf (\_TZ.THRM)) {
Notify (\_TZ.THRM, 0x81)
}
/* Update Secondary Thermal Zone */
- If (CondRefOf (\_TZ.SKIN, Local0)) {
+ If (CondRefOf (\_TZ.SKIN)) {
Notify (\_TZ.SKIN, 0x81)
}
}