[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Redo ACPIMMIO_BASE and _BANK

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42894 )

Change subject: soc/amd/common: Redo ACPIMMIO_BASE and _BANK
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42894/8/src/soc/amd/common/block/in... 
File src/soc/amd/common/block/include/amdblocks/acpimmio_map.h:

https://review.coreboot.org/c/coreboot/+/42894/8/src/soc/amd/common/block/in... 
PS8, Line 97: #ifdef __ACPI__
> A bit more background; all addresses here represent the x86-centric view of the memory map and shoul […]

ASL change CB:42944 does not build so the comment below still applies.

Memory32Fixed (ReadWrite, 0xfed81500, 0x400)
vs.
   Memory32Fixed (ReadWrite, (0xfed80000 + 0x1500), 0x400)