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

[Asami Doi (Code Review)]

Asami Doi 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 30:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/33387/29/src/mainboard/emulation/qe... 
PS29, Line 30: select
> Remove the comment.

Done

https://review.coreboot.org/c/coreboot/+/33387/29/src/mainboard/emulation/qe... 
PS29, Line 48: config DRAM_SIZE_MB
> qemu supports more than 512MiB of DRAM on that board, doesn't it? […]

If DRAM_SIZR_MB is larger than 1024 with "-m 2048M" as a QEMU option, an exception happens. I think that's because it tries to call read32()/write32() to beyond the QEMU memory size.
We can avoid the exception when we use "-m <large memory size>" on QEMU.