[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 18:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/33387/18/src/mainboard/emulation/qe... 
PS18, Line 20: static void mainboard_read_resources(struct device *dev)
looks good

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

https://review.coreboot.org/c/coreboot/+/33387/15/src/mainboard/emulation/qe... 
PS15, Line 9: void udelay(unsigned int n);
> Thank you for the reference. Sorry, I made a mistake and this file can be dropped. […]

Oh yeah, you are right. CNTFRQ is already set by qemu.