diff options
| author | Martin Roth <martinroth@google.com> | 2015-12-10 08:04:44 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2015-12-26 20:53:49 +0100 |
| commit | d890b45520a4b2af2ae2b15096b5897ed7464fcb (patch) | |
| tree | bf142bbca711b64adda6c421828f8ce16728c041 /src/ec | |
| parent | 80ef7b778e5e5ec8b2940c875df76bb705855ede (diff) | |
ACPI: Add hack to avoid IASL warning when reading back registers
Upcoming versions of IASL give a warning about unused methods. This
adds an operation after the read to use the local variable and avoid
the warning.
The warning can be completely disabled on the command line, but as it
can find real issues, my preference is to not do that.
Fixes warnings:
dsdt.aml 640: Store (CTMP, Local0)
Warning 3144 - Method Local is set but never used ^ (Local0)
Change-Id: If55bb8e03abb8861e1f2f08a8bcb1be8c9783afe
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12704
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/ec')
| -rw-r--r-- | src/ec/quanta/ene_kb3940q/acpi/ec.asl | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/ec/quanta/ene_kb3940q/acpi/ec.asl b/src/ec/quanta/ene_kb3940q/acpi/ec.asl index 8d3ff81992..251b4500ca 100644 --- a/src/ec/quanta/ene_kb3940q/acpi/ec.asl +++ b/src/ec/quanta/ene_kb3940q/acpi/ec.asl @@ -143,6 +143,9 @@ Device (EC0) // Force a read of CPU temperature Store (CTMP, Local0) + // Use Local0 to avoid iasl warning: Method Local is set but never used + And(Local0, Ones, Local0) + // Find and program number of P-States Store (SizeOf (\_PR.CP00._PSS), MPST) Store ("Programming number of P-states: ", Debug) |