Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36273 )
Change subject: lib/cbmem_top: Add a common cbmem_top implementation ......................................................................
Patch Set 16:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36273/13//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36273/13//COMMIT_MSG@16 PS13, Line 16: if CONFIG_RAMSTAGE_CBMEM_TOP_ARG is not set the : same implementation as will be used as in romstage. :
Hah. I pushed the same change as CB:36446. […]
Done
https://review.coreboot.org/c/coreboot/+/36273/13/src/lib/cbmem_top.c File src/lib/cbmem_top.c:
https://review.coreboot.org/c/coreboot/+/36273/13/src/lib/cbmem_top.c@22 PS13, Line 22: {
Sorry for nitpicking, but I think it's clearer if we do the following: […]
Done