On Sat, Mar 12, 2016 at 10:13:43PM +0100, Mateusz Kulikowski wrote:
On 12.03.2016 00:58, Daniel Glöckner wrote:
It is also confusing to have the "add" instruction in there without an explanation, especially because having it at offset 4 instead of 0 defeats its original purpose (MZ EXE signature for EFI).
My bad, I should have put it the other way around: add x13, x18, #0x16, b _start
Will do that for v3
The Dragonboard doesn't have an EFI BIOS, so we don't need that signature. I'd replace it with a ".long 0" (after the branch instruction of course), as it is done in Linux when CONFIG_EFI is not defined.
I know the goal is to eventually replace Little Kernel, but how about using the device tree passed by it? We could add some code to head.S that saves x0 in sp_el0 and abuse CONFIG_OF_HOSTFILE to retrieve it.
I prefer not to (at least for now) - It will not give us significant benefits (image will be a bit smaller and boot slightly faster, but it's not an issue as U-Boot replaces huge kernel image) and will just confuse people that already are using it. Is it fine with you?
No need to ask me. I'm not a regular on this list.
Best regards,
Daniel