[M] Change in coreboot[main]: acpi: Move HPET generation to a common location

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans, Cliff Huang, Felix Singer, Fred Reitberger, Jason Glenesk, Jeff Daly, Lance Zhao, Matt DeVillier, Raul Rangel, Tim Wawrzynczak, Vanessa Eusebio.

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76128?usp=email )

Change subject: acpi: Move HPET generation to a common location
......................................................................

Patch Set 9: Code-Review+1

(4 comments)

Patchset:

PS9: 
Looks good overall and I like it!

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/76128/comment/2e3c94ca_dc2d5253 :
PS9, Line 1266: #endif
A `static inline` with these guards in the header file at the forward
declaration might look better than #if in a C file.

File src/acpi/acpi_hpet.c:

https://review.coreboot.org/c/coreboot/+/76128/comment/134a514c_42532e9c :
PS9, Line 13: 		return;
It's already guarded in the Makefile, so not necessary here, right?
(I guess we could also keep this, remove the guard in the Makefile
and safe us the #if'ed stub above?)

File src/southbridge/intel/i82371eb/Kconfig:

https://review.coreboot.org/c/coreboot/+/76128/comment/16eed369_e5c1bbc8 :
PS9, Line 12: config ACPI_HPET
There's no I82371EB specific guard here, so this could be leaking, right?
Why not have a `select ACPI_HPET if HAVE_ACPI_TABLES` above?