[coreboot-gerrit] Change in coreboot[master]: soc/amd/common/block/acpimmio: fix ACPIMMIO decode enable function

[Michał Żygowski (Code Review)]

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37178 )

Change subject: soc/amd/common/block/acpimmio: fix ACPIMMIO decode enable function
......................................................................

Patch Set 8:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/37178/8/src/soc/amd/common/block/in... 
PS8, Line 39:  * in PMx24[1] (older implementations) and PMx04[1] (newer implementations).
> PMx24[0] ?

The comment is wrong. According to RRGs bit 0 enables the blocks (default disable) and bit 1 configures the access mode (IO mapped - 1; or default memory mapped - 0). Will update it.