[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Add restore and enable GPIO PM

[EricR Lai (Code Review)]

EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41712 )

Change subject: soc/intel/common: Add restore and enable GPIO PM
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41712/4/src/soc/intel/common/acpi/g... 
File src/soc/intel/common/acpi/gpio.asl:

https://review.coreboot.org/c/coreboot/+/41712/4/src/soc/intel/common/acpi/g... 
PS4, Line 22: Name(GPMB, Package(TOTAL_GPIO_COMM) {0})
> No error. And I can print it by Printf ("%o", GPMB(Local0)), totally fine though.

@Tim, here is the ACPI spec:

Example 3: This code allocates space for ten objects to be defined at runtime (see the Name and
Index term definitions).
Package (10) {}
Example 4: These package declarations will cause the compiler to emit a VarPackageOp AML
opcode.
Name (SIZE, 4)
Package (SIZE) {}
Package (1024) {}
Package (256) {}

This will be treated as runtime var, so I think I can remove the 0 and this won't confuse others.