On 12 September 2012 21:57, Tom Rini trini@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.kashyap@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?