On Sun, Jan 20, 2013 at 1:53 AM, Marek Vasut <ma...@denx.de> wrote:
> Dear Linus Walleij,
>
>> This, enabled the FDT library for the Integrators, updates
>> the Integrator/CP default command to load and pass a Device
>> Tree when booting the kernel from the on-board ethernet,
>> define same environment for the Integrator/AP and move the
>> load address around to something even.
>>
>> Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
>
> Shouldn't the load address be 0x8000 instead of 0x800 (it used to be 0x7fc0) ?

0x800 works fine, but honestly I don't know why Peter P ages ago chose the
very odd address 0x7fc0.

But you know U-boot way better than me, and most things work fine it seems.
What is a typical load address?

> Also, "setenv loadaddr nnnn" is nonsense, just use the usual
> CONFIG_SYS_LOAD_ADDR (which inherently sets the loadaddr env variable). Or do
> you see a problem there?

Not really, it was mainly because there is no way to set the FDT load
address as a variable, e.g. we don't have CONFIG_FDT_LOAD_ADDR,
and I wanted to keep both definitions in the same place, i.e. in the
default boot script.

On the Integrator/AP I have to load the two pieces one at the time,
manually using loady, so it is a bit helpful to have the address prints
at least.

Yours,
Linus Walleij
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to