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

[Jingle Hsu (Code Review)]

Attention is currently required from: Lance Zhao, Maxim Polyakov, Jonathan Zhang.
Jingle Hsu 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:

(1 comment)

Patchset:

PS41:
> > Maybe the problem is in the wrong GPIO Community 1 range only? […]

There is a mapping in Linux between resource array and barNo. 
In drivers/base/platform.c platform_get_resource(), it would look for 
the resource by the array index from the parameter 'unsigned int num'.
Resource array is declared as ResourceTemplate in gpio.asl.
barNo is the index of the resource to be returned.

In this case: LBG_COMMUNITY(5, 181, 246), 
kernel will look for resource through array index 5, if we don't add COM2 to ResourceTemplate(),
it will got the wrong resouce 'Interrupt' which causes returning error of invalid resource.