diff options
author | Martin Roth <martinroth@google.com> | 2015-12-08 15:26:03 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2015-12-10 16:30:50 +0100 |
commit | 10f7f5044ec8364419a2722b2822db3d9d94a7ed (patch) | |
tree | a3f64b67d1c73326a0e6905de4c3b14148c2ea10 | |
parent | fad23134009291a7b992c1323ca815421a4b557d (diff) | |
download | coreboot-10f7f5044ec8364419a2722b2822db3d9d94a7ed.tar.xz |
ACPI: Fix IASL Warning about unused method for GBUF check
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: Ie2f46808e92c309a63ba7661bcbd77402a08366a
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12694
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
-rw-r--r-- | src/soc/intel/baytrail/acpi/lpe.asl | 2 | ||||
-rw-r--r-- | src/soc/intel/braswell/acpi/lpe.asl | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/intel/baytrail/acpi/lpe.asl b/src/soc/intel/baytrail/acpi/lpe.asl index a8ebc3532d..1f13fe56e9 100644 --- a/src/soc/intel/baytrail/acpi/lpe.asl +++ b/src/soc/intel/baytrail/acpi/lpe.asl @@ -68,7 +68,7 @@ Device (LPEA) Store (\LPFW, BAS2) /* Append any Mainboard defined GPIOs */ - If (CondRefOf (^GBUF, Local0)) { + If (CondRefOf (^GBUF)) { ConcatenateResTemplate (^RBUF, ^GBUF, Local1) Return (Local1) } diff --git a/src/soc/intel/braswell/acpi/lpe.asl b/src/soc/intel/braswell/acpi/lpe.asl index b213fe6248..6dd73ab2e1 100644 --- a/src/soc/intel/braswell/acpi/lpe.asl +++ b/src/soc/intel/braswell/acpi/lpe.asl @@ -68,7 +68,7 @@ Device (LPEA) Store (\LPFW, BAS2) /* Append any Mainboard defined GPIOs */ - If (CondRefOf (^GBUF, Local0)) { + If (CondRefOf (^GBUF)) { ConcatenateResTemplate (^RBUF, ^GBUF, Local1) Return (Local1) } |