[coreboot-gerrit] [S] Change in coreboot[master]: soc/amd/*/root_complex: reserve IOMMU MMIO area

[Felix Held (Code Review)]

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

Change subject: soc/amd/*/root_complex: reserve IOMMU MMIO area
......................................................................

Patch Set 5:

(2 comments)

Patchset:

PS4:
> btw, quick look read_resources could be common code.

ideally the common code for that should be split into the x86 architecture specific, silicon specific and fsp integration specific parts. still need to look into that, but it's not a too high priority for me at the moment

PS4:
> I found the following in docs. I guess reserving the full 4G is a good idea. […]

yes, that's the part of the doc i looked at. the default address for the rom3 full spi flash mapping defaults to the same address; not sure why that is the case, but still seems to work.
instead of those 3 regions that cover most of the 4GB, i just reserved the full 4GB