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

[Arthur Heymans (Code Review)]

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 12:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36273/10/src/include/cbmem.h 
File src/include/cbmem.h:

https://review.coreboot.org/c/coreboot/+/36273/10/src/include/cbmem.h@78 
PS10, Line 78:  * and later stages. */
> Which I still hold is an incorrect assumption. But the code thus far handles it thanks to little endian on x86.

Updated it to say that the result of cbem_top_romstage has to fit in the size of uintptr_t in ramstage.

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

https://review.coreboot.org/c/coreboot/+/36273/10/src/lib/Kconfig@31 
PS10, Line 31: the
> as the?

Done