From 2883f7af94c51bc2105fc5e5447c1f61e5c357ee Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Mon, 13 Jan 2020 10:56:58 +0530 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/38385 Tested-by: build bot (Jenkins) Reviewed-by: Lance Zhao Reviewed-by: Angel Pons --- src/soc/intel/cannonlake/acpi/gpio.asl | 2 +- src/soc/intel/cannonlake/acpi/scs.asl | 10 +++------- 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 })