summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2018-05-06 21:08:37 -0500
committerMartin Roth <martinroth@google.com>2018-05-15 15:43:41 +0000
commit5dbe8ee7259d72ddfa2c273e2d8fca70addbf6bc (patch)
tree424d8c43d8ad6dcdadaf7dd7fae71d455f763835
parent782c910e867d37867be7459f7619155b24be35e7 (diff)
downloadcoreboot-5dbe8ee7259d72ddfa2c273e2d8fca70addbf6bc.tar.gz
coreboot-5dbe8ee7259d72ddfa2c273e2d8fca70addbf6bc.tar.bz2
coreboot-5dbe8ee7259d72ddfa2c273e2d8fca70addbf6bc.zip
ACPI: Set the correct number of arguments in ACPI methods
These methods had unused arguments and could be corrected by setting the correct number in the method initializer. Change-Id: I86606cfa1c391e2221cee31994e83667fa9ead61 Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/26125 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
-rw-r--r--src/drivers/intel/gma/acpi/common.asl2
-rw-r--r--src/mainboard/getac/p470/acpi/ec.asl2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/drivers/intel/gma/acpi/common.asl b/src/drivers/intel/gma/acpi/common.asl
index dd294fe93de4..7415859d5c96 100644
--- a/src/drivers/intel/gma/acpi/common.asl
+++ b/src/drivers/intel/gma/acpi/common.asl
@@ -97,7 +97,7 @@
}
/* Device Set State */
- Method(XDSS, 2)
+ Method(XDSS, 1)
{
/* If Parameter Arg0 is (1 << 31) | (1 << 30), the
* display switch was completed
diff --git a/src/mainboard/getac/p470/acpi/ec.asl b/src/mainboard/getac/p470/acpi/ec.asl
index 7420d4ae3f30..3c6cecfbbb9c 100644
--- a/src/mainboard/getac/p470/acpi/ec.asl
+++ b/src/mainboard/getac/p470/acpi/ec.asl
@@ -547,7 +547,7 @@ Scope(\_SB)
}
/* Set DKD */
- Method(SDKD, 1, Serialized)
+ Method(SDKD, 0, Serialized)
{
TRAP(0xda)
Return (0)