[M] Change in coreboot[master]: soc/amd/common/data_fabric/domain: provide amd_pci_domain_fill_ssdt

[Felix Held (Code Review)]

Attention is currently required from: Arthur Heymans, Fred Reitberger, Jason Glenesk, Matt DeVillier, Nico Huber, Raul Rangel.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74843?usp=email )

Change subject: soc/amd/common/data_fabric/domain: provide amd_pci_domain_fill_ssdt
......................................................................

Patch Set 14:

(1 comment)

File src/soc/amd/common/block/data_fabric/domain.c:

https://review.coreboot.org/c/coreboot/+/74843/comment/17b96ed2_193f805a :
PS11, Line 169: 	if (base > PCI_IO_CONFIG_LAST_PORT || limit < PCI_IO_CONFIG_INDEX) {
> read that part of the spec and it looks like a dword io producer resource should just work. […]

tested with https://review.coreboot.org/c/coreboot/+/75613/2 applied which i then removed form the patch train du to it not working properly in the linux case.

windows 10 doesn't care if it's a word or dword io resource type; the kernel from ubuntu 2022.4 doesn't like the dword io resource type much though:

acpi PNP0A08:00: host bridge window expanded to [io  0x0000-0xffff]; [io  0x0000-0xffff window] ignored

at least it doesn't fail like in the case with the length field being truncated to 0