diff options
author | Martin Roth <martinroth@google.com> | 2015-11-23 12:37:22 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2015-11-24 22:42:57 +0100 |
commit | 601ea3d40e39bb528f6418757ae3e8482920198f (patch) | |
tree | 218edcde825fce6187ce18573e7099146f5edadb /src/mainboard/lenovo/t400/acpi/graphics.asl | |
parent | c5fb1a2ea8925cad750183c16a03214e43961607 (diff) |
lenovo t400: Fix IASL warning and remark
If any path in a method returns a value, IASL expects that all paths
within that method will return a value.
Presumably, the ATPX would not need a return value if Arg0 is anything
other than 0, so just return a zero.
- Serialize ATPX method to make IASL happy. This means that it can
only be used by one thread at a time.
Fixes these issues:
dsdt.aml 2581: Method (ATPX, 2, NotSerialized) {
Remark 2120 - ^ Control Method should be made Serialized
(due to creation of named objects within)
dsdt.aml 2581: Method (ATPX, 2, NotSerialized) {
Warning 3115 - ^ Not all control paths return a value (ATPX)
Change-Id: I14aeab0cebe4596e06a17cffc36cc01b953d7191
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12518
Reviewed-by: Timothy Pearson <tpearson@raptorengineeringinc.com>
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/mainboard/lenovo/t400/acpi/graphics.asl')
-rw-r--r-- | src/mainboard/lenovo/t400/acpi/graphics.asl | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mainboard/lenovo/t400/acpi/graphics.asl b/src/mainboard/lenovo/t400/acpi/graphics.asl index d2cfdb93c2..818ea93724 100644 --- a/src/mainboard/lenovo/t400/acpi/graphics.asl +++ b/src/mainboard/lenovo/t400/acpi/graphics.asl @@ -55,7 +55,7 @@ Method(SHYB, 1) { } } -Method (ATPX, 2, NotSerialized) { +Method (ATPX, 2, Serialized) { /* Create local variables */ Name (ATPR, Buffer (0x08) { 0x0, 0x0, 0x0, 0x0, @@ -94,4 +94,5 @@ Method (ATPX, 2, NotSerialized) { SHYB(0x00) } } + Return(Zero) } |