[S] Change in coreboot[main]: soc/intel/common/block/acpi: Add GPE1 blocks to ACPI FADT table

[Subrata Banik (Code Review)]

Attention is currently required from: Anil Kumar K, Bora Guvendik, Cliff Huang, Hannah Williams, Paul Menzel.

Subrata Banik has posted comments on this change by Cliff Huang. ( https://review.coreboot.org/c/coreboot/+/84103?usp=email )

Change subject: soc/intel/common/block/acpi: Add GPE1 blocks to ACPI FADT table
......................................................................

Patch Set 10:

(2 comments)

File src/soc/intel/common/block/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/84103/comment/8eb53f0b_1a369903?usp... :
PS7, Line 121: 	if (CONFIG(SOC_INTEL_COMMON_BLOCK_ACPI_USE_GPE1)) {
> Subrata,
>
> agree. We will need only _HAVE_GPE1 flag as long as the SOC header uses _HVAE_GPE1 flag for providing GPE1 defines. For the case of not using GPE1 in PTL, we will unset this flag.

yes, that makes more sense to me. but note if you wish to make `_HAVE_GPE1` Kconfig configurable then better to select from mainboard and not SoC default.

https://review.coreboot.org/c/coreboot/+/84103/comment/e5fac0a3_01b563a3?usp... :
PS7, Line 128:  8
> yes.

Acknowledged