[U-Boot] [PATCH v4] PXE: FDT: Add support for fdt in PXE

[Chander Kashyap]

On 12 September 2012 21:57, Tom Rini <trini at 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 at 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