On 12 September 2012 21:57, Tom Rini <tr...@ti.com> wrote: > On Fri, Sep 07, 2012 at 11:06:31AM +0530, Chander Kashyap wrote: > >> Now DT support is becoming common for all new SoC's. Hence it is better >> to have option for getting specific FDT from the remote server. >> >> This patch adds support for new label i.e. 'fdt'. This will allow to >> retrieve 'fdt blob' from the remote server. This patch take care for >> the following scenarios. >> >> The usage of fdt is optional. >> The 'fdt blob' can be retrieved from tftp or can be available locally >> or can be absent. >> >> If 'fdt_addr_r' environment variable is set and 'fdt' label is defined >> retrieve 'fdt blob' from tftp. 'fdt_addr_r' is then passed along bootm >> command. >> >> If 'fdt_addr' is set and 'fdt blob' is not retrieved from the tftp pass >> 'fdt_addr' to bootm command. In this case 'fdt blob' will be available >> at 'fdt_addr'. >> >> If 'fdt_addr' is not set and 'fdt blob' is not retrieve from tftp pass >> NULL to boot command. In this case 'fdt blob' is not required and absent. >> >> Signed-off-by: Chander Kashyap <chander.kash...@linaro.org> >> --- >> Changes in v2: >> - Removed the duplicate code. >> changes in v3: >> - Added documentation for "fdt" lable in doc/README.pxe >> changes in v4: >> - Added New environment variable 'fdt_addr_r' for 'fdt blob' >> - Add more descriptive documentation for the 'fdt' retrieval. >> >> common/cmd_pxe.c | 39 ++++++++++++++++++++++++++++++++++++--- >> doc/README.pxe | 14 ++++++++++++-- >> 2 files changed, 48 insertions(+), 5 deletions(-) > > Looks good to me, Joe can you please pick this up if you're good with > it? Thanks! > > -- > Tom
Any feedback on this? -- with warm regards, Chander Kashyap _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
