diff options
| author | Martin Roth <martinroth@google.com> | 2015-11-23 12:23:19 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2015-11-24 22:34:34 +0100 |
| commit | 21dbc2fc3c2febe0e6ac3760f4c8a63c2aeedd06 (patch) | |
| tree | 48a8cb3e9fd9e2278f575b3d8d458993b7ba753e /src/ec/lenovo/h8/acpi | |
| parent | 56033a9f2d6ceed6ad27450d866be46eb49abf8b (diff) | |
ec/lenovo/h8: Fix IASL warnings
If any path in a method returns a value, IASL expects that all paths
within that method will return a value.
Presumably the MKHP method wouldn't get called unless there were a
pending event, but if no event is found, return a zero.
Fixes IASL warning:
dsdt.aml 1785: Method (MHKP, 0, NotSerialized)
Warning 3115 - ^ Not all control paths return a value (MHKP)
This was the only IASL warning in most lenovo mainboards.
Change-Id: Id93dcc4a74bd4c18b78f1dde821e7ba0f3444da3
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12517
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/ec/lenovo/h8/acpi')
| -rw-r--r-- | src/ec/lenovo/h8/acpi/ec.asl | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/ec/lenovo/h8/acpi/ec.asl b/src/ec/lenovo/h8/acpi/ec.asl index d1789e6e39..085492d539 100644 --- a/src/ec/lenovo/h8/acpi/ec.asl +++ b/src/ec/lenovo/h8/acpi/ec.asl @@ -300,6 +300,7 @@ Device(EC) Add (Local0, 0x5000, Local0) Return (Local0) } + Return (Zero) } /* Report event */ Method (RHK, 1, NotSerialized) { |