Re: [PATCH v2 13/25] binman: Make fake blobs zero-sized by default

Alper Nebi Yasak
2022-03-03T21:17:56Z

On 24/02/2022 02:00, Simon Glass wrote:
> On x86 devices having even a small amount of data can cause an overlap
> between regions. For example, bayleybay complains when the intel-vga
> region overlaps with u-boot-ucode:
>
> ImagePos Offset Size Name
> <none> 00000000 00800000 main-section
> <none> ff800000 00000080 intel-descriptor
> <none> ff800400 00000080 intel-me
> <none> fff00000 00098f24 u-boot-with-ucode-ptr
> <none> fff98f24 00001aa0 u-boot-dtb-with-ucode
> <none> fff9a9d0 0002a000 u-boot-ucode
> <none> fffb0000 00000080 intel-vga
> ...
>
> It is safer to use an empty file in most cases. Add an option to set the
> size for those uses that need it.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> Changes in v2:
> - Add a patch to make fake blobs zero-sized by default
>
> tools/binman/entry.py | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
> diff --git a/tools/binman/entry.py b/tools/binman/entry.py

Reviewed-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>

Later on, I think adding 'min-size' and 'max-size' properties for
entries could be useful for a few things including this.