On 16 July 2015 at 10:34, Simon Glass <s...@chromium.org> wrote:
> On 16 July 2015 at 07:26, Linus Walleij <linus.wall...@linaro.org> wrote:
>> On Fri, Jul 10, 2015 at 6:18 PM, Sudeep Holla <sudeep.ho...@arm.com> wrote:
>>
>>> Commit 90fbee3e4051 ("cmd_fdt: Actually fix fdt command in sandbox")
>>> changed the format(from hex address to unsigned long) in which "fdtaddr"
>>> is saved . However do_fdt continues reads the "fdtaddr" assuming it to
>>> be in hex format. This may lead to fdt being either loaded or attempted
>>> to load at erroneous address generating fault if the address is out of
>>> memory.
>>>
>>> This patch changes back the format to hex while saving the "fdtaddr"
>>> as it was done before.
>>>
>>> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com>
>>> Cc: Linus Walleij <linus.wall...@linaro.org>
>>> Cc: Tom Rini <tr...@konsulko.com>
>>> Cc: Simon Glass <s...@chromium.org>
>>> Cc: Hua Yanghao <huayang...@gmail.com>
>>> Cc: Heiko Schocher <h...@denx.de>
>>
>> Looks like the right fix:
>> Acked-by: Linus Walleij <linus.wall...@linaro.org>
>>
>> Yours,
>> Linus Walleij
>
> Acked-by: Simon Glass <s...@chromium.org>Applied to u-boot-fdt, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
