[coreboot-gerrit] Change in coreboot[master]: cpu/amd/pi: Don't redefine XIP_ROM_SIZE

[Arthur Heymans (Code Review)]

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

Change subject: cpu/amd/pi: Don't redefine XIP_ROM_SIZE
......................................................................

Patch Set 1:
> Patch Set 1:
>
> I don't see XIP_ROM_SIZE referenced with AGESA or binaryPI anywhere, amd_initmmio() sets CACHE_ROM_SIZE as WRPROT early in romstage. Looks like you can remove them all and select NO_FIXED_XIP_ROM_SIZE.
>
> Only non-AGESA AMD uses cpu/amd/car/ where you see CONFIG_XIP_ROM_SIZE.

ack.