Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37402 )
Change subject: soc/amd/common/blocks/acpimmio: extend the block with BIOSRAM storage ......................................................................
Patch Set 3:
(2 comments)
Patch Set 1:
(2 comments)
You can combine new ramtop.c and uma.c and make it acpimmio/biosram.c.
Done. Merged it into ACPIMMIO and squashed with CB:37402.
https://review.coreboot.org/c/coreboot/+/37402/1/src/soc/amd/common/block/in... File src/soc/amd/common/block/include/amdblocks/biosram_layout.h:
https://review.coreboot.org/c/coreboot/+/37402/1/src/soc/amd/common/block/in... PS1, Line 21:
Bring prototypes from uma.h here and remove that file. You can remove _layout from filename too.
Done
https://review.coreboot.org/c/coreboot/+/37402/1/src/soc/amd/common/block/ra... File src/soc/amd/common/block/ramtop/Kconfig:
https://review.coreboot.org/c/coreboot/+/37402/1/src/soc/amd/common/block/ra... PS1, Line 1: config SOC_AMD_COMMON_BLOCK_RAMTOP
I am pretty sure we can do with just SOC_AMD_COMMON_BLOCK_BIOSRAM. […]
I just merged it with ACPIMMIO.