[coreboot-gerrit] Change in coreboot[master]: intel/xeon_sp: add ACPI to control GPIO

[Maxim Polyakov (Code Review)]

Attention is currently required from: Lance Zhao, Jonathan Zhang, Jingle Hsu.
Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39979 )

Change subject: intel/xeon_sp: add ACPI to control GPIO
......................................................................

Patch Set 44:

(2 comments)

Patchset:

PS41:
> The current kernel is the following, why we need to com2 to be included? Because of calculation? […]

@Lance, I agree with you, at the beginning I excluded com2 according to this table from the kernel sources, but testing shows that this is necessary.

@Jingle, thanks for testing. I'm glad it finally works, but are you sure about com2? Maybe the problem is in the wrong GPIO Community 1 range only?

File src/soc/intel/xeon_sp/acpi/gpio.asl:

https://review.coreboot.org/c/coreboot/+/39979/comment/e81f8a7e_4d7ea4fc 
PS43, Line 13: 1
> If we have only one "INT35336" and then _UID is not needed or _UID can be 0.

Done