[coreboot-gerrit] Change in coreboot[master]: amd/agesa: Make BottomIo position configurable

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38472 )

Change subject: amd/agesa: Make BottomIo position configurable
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38472/7/src/northbridge/amd/agesa/f... 
File src/northbridge/amd/agesa/family16kb/state_machine.c:

https://review.coreboot.org/c/coreboot/+/38472/7/src/northbridge/amd/agesa/f... 
PS7, Line 23: MIN(0xE0,
            : 			MAX(0x28, ((UINT16)(CONFIG_BOTTOMIO_POSITION >> 24) & 0xF8)));
It's a bit hard to read with magic numbers.
I suggest (UINT16)(MIN(0xe0000000, MAX(0x28000000, CONFIG_BOTTOMIO_POSITION)) >> 24) & 0xf8;