On Sun, Jan 20, 2013 at 1:53 AM, Marek Vasut marex@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.walleij@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