GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Subrata Banik 2020-01-13 10:56:58 +05:30
parent 159cd3f421
commit 2883f7af94
2 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/soc/intel/cannonlake/acpi/gpio.asl
View File

@ -159,7 +159,7 @@ Method (CGPM, 2, Serialized)
} }
/* GPIO Power Management bits */ /* 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 * Save GPIO Power Management bits

10
src/soc/intel/cannonlake/acpi/scs.asl
View File

@ -107,8 +107,6 @@ Scope (\_SB.PCI0) {
If (LEqual (Arg0, ^DSUU)) { If (LEqual (Arg0, ^DSUU)) {
/* Check the revision */ /* Check the revision */
If (LGreaterEqual (Arg1, Zero)) { If (LGreaterEqual (Arg1, Zero)) {
/* Switch statement based on the function index. */
Switch (ToInteger (Arg2)) {
/* /*
* Function Index 0 the return value is a buffer containing * Function Index 0 the return value is a buffer containing
* one bit for each function index, starting with zero. * one bit for each function index, starting with zero.
@ -125,10 +123,9 @@ Scope (\_SB.PCI0) {
* For SD we have to support functions to * For SD we have to support functions to
* set 1.8V signalling and 3.3V signalling [BIT4, BIT3] * set 1.8V signalling and 3.3V signalling [BIT4, BIT3]
*/ */
Case (0) { If (LEqual (Arg2, Zero)) {
Return (Buffer () { 0x19 }) Return (Buffer () { 0x19 })
} }
/* /*
* Function Index 3: Set 1.8v signalling. * Function Index 3: Set 1.8v signalling.
* We put a sleep of 100ms in this method to * 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 * UHS SD card on PCH. This is to compensate
* for the SD VR slowness. * for the SD VR slowness.
*/ */
Case (3) { If (LEqual (Arg2, 3)) {
Sleep (100) Sleep (100)
Return(Buffer () { 0x00 }) Return(Buffer () { 0x00 })
} }
@ -147,11 +144,10 @@ Scope (\_SB.PCI0) {
* UHS SD card on PCH. This is to compensate * UHS SD card on PCH. This is to compensate
* for the SD VR slowness. * for the SD VR slowness.
*/ */
Case (4) { If (LEqual (Arg2, 4)) {
Sleep (100) Sleep (100)
Return(Buffer () { 0x00 }) Return(Buffer () { 0x00 })
} }
}
} }
} }
Return(Buffer() { 0x0 }) Return(Buffer() { 0x0 })