On Tue, Oct 2, 2012 at 5:26 AM, Stefan Roese sr@denx.de wrote:
On 09/27/2012 10:43 PM, McClintock Matthew-B29882 wrote:
By calling fdt_chosen(), the chosen node will be updated / created by the "fdt boardsetup" command. This is useful for setting of the kernel commandline via the "bootargs" env variable.
With this change, the "fdt boardsetup" can be used to prepare the DT blob for SPL booting. The patched DT blob can be saved to flash and can be used by the SPL U-Boot version directly for Linux booting.
Signed-off-by: Stefan Roese sr@denx.de
common/cmd_fdt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
There are possibly some workflows this breaks. I can't remember if for AMP boot we need to do something between ft_board_setup() and fdt_chosen()
Could you please elaborate what exactly you fear here? Is this a NACK for this patch?
Possibly. I've got to find our docs on how we do 2-core AMP booting sequence w/regards to device tree.
See if this works:
http://www.freescale.com/infocenter/index.jsp?topic=%2FQORIQSDK%2F2151493.ht...
Kumar, did you find the time to check/test this? I would really like to know if this patch is okay with you.
This should be fine. You are calling fdt_chosen(), but we can still run 'fdt chosen $initrd_start $initrd_end' after 'fdt boardsetup'. A quick glance at fdt_chosen() suggest it can be run multiple times without error.
You might add a comment in the U_BOOT_CMD that mentions boardsetup calls chosen.
-M