[coreboot-gerrit] Change in coreboot[master]: util/cbfstool: Add ARM64 Image support

[Julius Werner (Code Review)]

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.


-- 
To view, visit https://review.coreboot.org/23869
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a8183a87de4dc3a4c4f038fe5b4912ada5d75f9
Gerrit-Change-Number: 23869
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Rudolph <patrick.rudolph at 9elements.com>
Gerrit-Reviewer: Julius Werner <jwerner at chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Tue, 27 Feb 2018 01:59:51 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180227/2e3d4762/attachment.html>