Julius Werner has posted comments on this change. ( https://review.coreboot.org/23869 )
Change subject: util/cbfstool: Add ARM64 Image support ......................................................................
Patch Set 2:
I'm a little confused why we need a trampoline and relocation code just to boot an ARM64 kernel. ARM64 kernels always want to be loaded at the start of physical memory (+0x80000). ARM64 boards in coreboot can statically position their ramstage wherever they want in memlayout. Why not just make sure that the ramstage is always far enough away from the start of physical memory to avoid the possibility of an overlap and be done with it? I mean, all relevant systems have more than enough DRAM available.
This just seems to add loads of complication to the process with no clear gain.
-
Julius Werner (Code Review)