[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/acpi: fix Windows crash on S0ix-enabled boards

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47138 )

Change subject: soc/intel/common/acpi: fix Windows crash on S0ix-enabled boards
......................................................................

Patch Set 1:

(3 comments)

https://review.coreboot.org/c/coreboot/+/47138/1/src/soc/intel/common/acpi/l... 
File src/soc/intel/common/acpi/lpit.asl:

https://review.coreboot.org/c/coreboot/+/47138/1/src/soc/intel/common/acpi/l... 
PS1, Line 37: 					Return(Package(5) {0, Ones, Ones, Ones, Ones})
This looks very odd. But I guess you also tried to return an empty package?

https://review.coreboot.org/c/coreboot/+/47138/1/src/soc/intel/common/acpi/l... 
PS1, Line 26: This function
What function?

https://review.coreboot.org/c/coreboot/+/47138/1/src/soc/intel/common/acpi/l... 
PS1, Line 26: debugging
Why would there be constraints for debugging?