[U-Boot] [PATCH v3] common/image.c: align usage of fdt_high with initrd_high

[Simon Glass]

Hi Stefan,

On Wed, Feb 8, 2012 at 3:39 AM, Stefano Babic <sbabic at denx.de> wrote:
> On 10/01/2012 08:54, Shawn Guo wrote:
>> The commit message of a28afca (Add uboot "fdt_high" enviroment variable)
>> states that fdt_high behaves similarly to the existing initrd_high.
>> But fdt_high actually has an outstanding difference from initrd_high.
>> The former specifies the start address, while the later specifies the
>> end address.
>>
>> As fdt_high and initrd_high will likely be used together, it'd be nice
>> to have them behave same.  The patch changes the behavior of fdt_high
>> to have it aligned with initrd_high.
>>
>> The document of fdt_high in README is updated with an example to
>> demonstrate the usage of this environment variable.
>>
>> Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
>> ---
>
> Hi,
>
> this patch seems to be required to test device tree on i.MX platform.
>
> Simon, I have seen several patches from you regarding fdt - maybe you
> can add your ACK or NACK to this patch.

I'm not familiar with this code, but will take a look. Also copying Jerry.

Regards,
Simon

>
> Stefano
>
> --
> =====================================================================
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
> =====================================================================