Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36334 )
Change subject: cbmem.h: Align comment with the reality of implementations ......................................................................
cbmem.h: Align comment with the reality of implementations
cbmem_top() should simply not be called before memory is initialed, in order for the implementation to return something meaningful.
Change-Id: I8fe32844af290626a0f91279143fda4d3442680f Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/include/cbmem.h 1 file changed, 2 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/36334/1
diff --git a/src/include/cbmem.h b/src/include/cbmem.h index f972ba6..4005fa2 100644 --- a/src/include/cbmem.h +++ b/src/include/cbmem.h @@ -71,9 +71,8 @@ /* Return the top address for dynamic cbmem. The address returned needs to * be consistent across romstage and ramstage, and it is required to be * below 4GiB for 32bit coreboot builds. On 64bit coreboot builds there's no - * upper limit. - * x86 boards or chipsets must return NULL before the cbmem backing store has - * been initialized. */ + * upper limit. This should not be called before memory is initialized. + */ void *cbmem_top(void);
/* Add a cbmem entry of a given size and id. These return NULL on failure. The