[coreboot-gerrit] Change in coreboot[master]: arch/arm: Pass cbmem_top to ramstage via calling argument

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36145 )

Change subject: arch/arm: Pass cbmem_top to ramstage via calling argument
......................................................................

Patch Set 15:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36145/15/src/arch/arm/stages.c 
File src/arch/arm/stages.c:

https://review.coreboot.org/c/coreboot/+/36145/15/src/arch/arm/stages.c@32 
PS15, Line 32: uintptr_t _cbmem_top_ptr;
> > Why can't the x86 stages do this as well? Why put it in assembler? […]

It doesn't need a similar function. It just needs to set the variable. You are allocating and setting the variable in the assembly when you only need the latter. The cbmem_top.c compilation unit can hold the object itself, right?