[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.
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 41:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/39979/comment/34355021_5463efff 
PS40, Line 126: 		Case (COMM_2) {
              : 			Local0 = PID_GPIOCOM2
> Community 3? Actually Community 2 is listed in EDS. […]

Do you mean the order of the communities in the pincntr driver?

https://github.com/torvalds/linux/blob/master/drivers/pinctrl/intel/pinctrl-...
static const struct intel_community lbg_communities[] = {
    LBG_COMMUNITY(0, 0, 71),
    LBG_COMMUNITY(1, 72, 132),
    LBG_COMMUNITY(3, 133, 144),
    LBG_COMMUNITY(4, 145, 180),
    LBG_COMMUNITY(5, 181, 246),
};