[coreboot-gerrit] Change in coreboot[master]: mainboard/emulation/qemu-aarch64: Add new board for ARMv8

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33387 )

Change subject: mainboard/emulation/qemu-aarch64: Add new board for ARMv8
......................................................................

Patch Set 15:

(2 comments)

https://review.coreboot.org/c/coreboot/+/33387/15/src/mainboard/emulation/qe... 
File src/mainboard/emulation/qemu-aarch64/mainboard.c:

https://review.coreboot.org/c/coreboot/+/33387/15/src/mainboard/emulation/qe... 
PS15, Line 1: /*
> arm64/romstage.c main() calls cbmem_initialize_empty(), that should be enough. […]

I wasn't aware of that, thanks.

https://review.coreboot.org/c/coreboot/+/33387/15/src/mainboard/emulation/qe... 
PS15, Line 29: 	cbmem_recovery(0);
> Why is this here? main() from hardwaremain.c calls cbmem_initialize(), that should be all you need.

I guess it's copied from another emulated board, doing it wrong, too.