[U-Boot] [PATCH PATCH v4 04/15] spl: fit: allocate a temporary buffer to load the overlays

[Jean-Jacques Hiblot]

Hi Simon,

On 17/09/2019 07:48, Simon Glass wrote:
> Hi Jean-Jacques,
>
> On Wed, 11 Sep 2019 at 06:32, Jean-Jacques Hiblot <jjhiblot at ti.com> wrote:
>> Hi Simon,
>>
>> On 13/08/2019 11:33, Simon Glass wrote:
>>> Hi Jean-Jacques,
>>>
>>> On Mon, 5 Aug 2019 at 03:44, Jean-Jacques Hiblot <jjhiblot at ti.com> wrote:
>>>> If the node describing an overlay does not specify a load address, it will
>>>> be loaded at the address previously used.
>>>> Fixing it by allocating a temporary 64kB region that will be used as a
>>>> default load address.
>>> How did you come to decide on 64KB? Might this be too large or too small?
>> It seemed big enough to me to accommodate a DTB overlay.
>>
>> There is no easy way to know how big the buffer must be without reading
>> it first because it can be compressed.
> You should be able to check the header to see the uncompressed size.

I looked into that, and there is no uncompressed size in the GZIP header.

We cannot know the size in advance.

JJ

>
> Regards,
> Simon
>