[U-Boot] [PATCH v2 17/18] board: Add Qualcomm Dragonboard 410C support

[Daniel Glöckner]

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

-- 
Dipl.-Math. Daniel Glöckner, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11,
Bertha-von-Suttner-Straße 9, 37085 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Dr. Uwe Kracke, Ust-IdNr.: DE 205 198 055

emlix - your embedded linux partner