[U-Boot] [PATCH v2 4/7] x86: ifdtool: Check that U-Boot does not overlap other regions

[Simon Glass]

On 19 August 2015 at 08:33, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Sun, Aug 16, 2015 at 4:37 AM, Simon Glass <sjg at chromium.org> wrote:
>> Since U-Boot and its device tree can grow we should check that it does not
>> overlap the regions above it. Track the ROM offset that U-Boot reaches and
>> check that other regions (written after U-Boot) do not interfere.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>
> Nice enhancement! I wanted to add similar feature when I was debugging
> the TunnelCreek graphics vgabios (sometimes it got overlapped sections
> in the u-boot.rom without any warnings), but never started to work on
> it. Thanks for the patch.

Yes it can silently bite you, so I thought it was a good idea to fix it.

>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
> Tested-by: Bin Meng <bmeng.cn at gmail.com>
>
> Regards,
> Bin

Applied to u-boot-x86.