[M] Change in coreboot[master]: UNTESTED acpi: Add ECAM region below PNP0C02 device in SSDT

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Felix Held, Jeff Daly, Lance Zhao, Matt DeVillier, Sean Rhodes, Tim Wawrzynczak, Vanessa Eusebio.

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

Change subject: UNTESTED acpi: Add ECAM region below PNP0C02 device in SSDT
......................................................................

Patch Set 2:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/75729/comment/07655781_e42859a7 :
PS2, Line 28: unknown reason
Probably because it meddles with e820. So far a reservation there
with the exact same base was enough to make Linux happy, FWIW.

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/75729/comment/9da23d98_019dab31 :
PS2, Line 587: 	struct device *dev = dev_find_path(NULL, DEVICE_PATH_DOMAIN);
             : 	assert(dev != NULL);
We could also export domain devices to `build/static_devices.h`.

OTOH, filtering the right device in the loop device should do too.

https://review.coreboot.org/c/coreboot/+/75729/comment/930241f1_4c752edf :
PS2, Line 594: 1
Shouldn't this be unique (I'm not sure about the scope)? i.e. not the same
as in the static `.asl` files.