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

[Lance Zhao (Code Review)]

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

(2 comments)

Patchset:

PS41:
> What I meant is that the controller file is not created: […]

The current kernel is the following, why we need to com2 to be included? Because of calculation?

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),
};

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

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