On Sep 26, 2012, at 9:20 AM, Stefan Roese wrote: > Hi Kumar, > > On 09/26/2012 03:36 PM, Kumar Gala 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 <s...@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. - k _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
