[U-Boot] [PATCH] cmd_elf: CONFIG_ELF_SIMPLE_LOAD: load ELFs according to PHDRs

[Mike Frysinger]

On Friday 05 December 2008 17:21:27 Wolfgang Denk wrote:
> In message <1225603202-5757-1-git-send-email-vapier at gentoo.org> you wrote:
> > The current ELF loading function does a lot of work above and beyond a
> > simple "loading".  It ignores the real load addresses and loads things
> > into their virtual (runtime) address.  This is undesirable when we just
> > want it to load an ELF and let the ELF do the actual C runtime init.
>
> I apologize for the late review...
>
> > +#ifdef CONFIG_ELF_SIMPLE_LOAD
> > +unsigned long load_elf_image (unsigned long addr)
>
> ...
>
> ...but would it not make sense to allow to make the decision which
> behaviour fits the user's requirements at runtime instead of compile
> time?
>
> We could either make this depend on some environment variable setting,
> or on a command line option when running the command.
>
>
> If you prefer to stick with the CONFIG_ELF_SIMPLE_LOAD solution, then
> this variable has to be documented in the README.

it doesnt matter to me which route is taken so long as the default can be 
controlled (so via env is ok as you can set the default env values).  if you 
prefer one over the other, let me know.  otherwise simply updating the README 
is the easiest route for me ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20081205/9ba137e8/attachment.pgp