[coreboot-gerrit] Change in coreboot[master]: lib/cbmem_top: Add a common cbmem_top implementation

[Julius Werner (Code Review)]

Julius Werner 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 9: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/36273/9/src/lib/Kconfig 
File src/lib/Kconfig:

https://review.coreboot.org/c/coreboot/+/36273/9/src/lib/Kconfig@27 
PS9, Line 27: config RAMSTAGE_CBMEM_TOP_ARG
nit: Does this need a new Kconfig or can we just make the default cbmem_top() implementation __weak and add it unconditionally? (I like to avoid adding boilerplate Kconfigs that every platform from now on will want to select.)

Alternatively, maybe we could turn the logic around and make a CUSTOM_CBMEM_TOP option that all current platforms need to select?