summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorSubrata Banik <subrata.banik@intel.com>2020-01-13 10:56:58 +0530
committerSubrata Banik <subrata.banik@intel.com>2020-01-14 07:26:19 +0000
commit2883f7af94c51bc2105fc5e5447c1f61e5c357ee (patch)
tree74462b77a3550a83168cfc2de3b1e58979579738 /src/soc
parent159cd3f42144c196e19d7d497085b51d0f4cd52c (diff)
downloadcoreboot-2883f7af94c51bc2105fc5e5447c1f61e5c357ee.tar.xz
soc/intel/cannonlake: Fix ASL compilation remarks
This patch fixes below ASL compilation remarks 1. dsdt.asl 495: Method (_DSM, 4) Remark 2119 - ^ Control Method marked Serialized (Due to use of Switch operator) 2. dsdt.asl 721: Name(GPMB, Package(5) {0}) Remark 2063 - ^ Initializer list shorter than declared package length Change-Id: Iabd6c39025713dda7aa69cb479f003fbec8855b3 Signed-off-by: Subrata Banik <subrata.banik@intel.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/38385 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Lance Zhao <lance.zhao@gmail.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/cannonlake/acpi/gpio.asl2
-rw-r--r--src/soc/intel/cannonlake/acpi/scs.asl10
2 files changed, 4 insertions, 8 deletions
diff --git a/src/soc/intel/cannonlake/acpi/gpio.asl b/src/soc/intel/cannonlake/acpi/gpio.asl
index 7702ad556c..e2eb78c48d 100644
--- a/src/soc/intel/cannonlake/acpi/gpio.asl
+++ b/src/soc/intel/cannonlake/acpi/gpio.asl
@@ -159,7 +159,7 @@ Method (CGPM, 2, Serialized)
}
/* GPIO Power Management bits */
-Name(GPMB, Package(TOTAL_GPIO_COMM) {0})
+Name(GPMB, Package(TOTAL_GPIO_COMM) {0, 0, 0, 0, 0})
/*
* Save GPIO Power Management bits
diff --git a/src/soc/intel/cannonlake/acpi/scs.asl b/src/soc/intel/cannonlake/acpi/scs.asl
index 0012a4dfd8..4938081f27 100644
--- a/src/soc/intel/cannonlake/acpi/scs.asl
+++ b/src/soc/intel/cannonlake/acpi/scs.asl
@@ -107,8 +107,6 @@ Scope (\_SB.PCI0) {
If (LEqual (Arg0, ^DSUU)) {
/* Check the revision */
If (LGreaterEqual (Arg1, Zero)) {
- /* Switch statement based on the function index. */
- Switch (ToInteger (Arg2)) {
/*
* Function Index 0 the return value is a buffer containing
* one bit for each function index, starting with zero.
@@ -125,10 +123,9 @@ Scope (\_SB.PCI0) {
* For SD we have to support functions to
* set 1.8V signalling and 3.3V signalling [BIT4, BIT3]
*/
- Case (0) {
+ If (LEqual (Arg2, Zero)) {
Return (Buffer () { 0x19 })
}
-
/*
* Function Index 3: Set 1.8v signalling.
* We put a sleep of 100ms in this method to
@@ -136,7 +133,7 @@ Scope (\_SB.PCI0) {
* UHS SD card on PCH. This is to compensate
* for the SD VR slowness.
*/
- Case (3) {
+ If (LEqual (Arg2, 3)) {
Sleep (100)
Return(Buffer () { 0x00 })
}
@@ -147,11 +144,10 @@ Scope (\_SB.PCI0) {
* UHS SD card on PCH. This is to compensate
* for the SD VR slowness.
*/
- Case (4) {
+ If (LEqual (Arg2, 4)) {
Sleep (100)
Return(Buffer () { 0x00 })
}
- }
}
}
Return(Buffer() { 0x0 })