On Mon, Aug 11, 2025 at 12:45:13PM +0200, Casey Connolly wrote:
> In 6e4675b8e5d8 ("lmb: replace the lmb_alloc() and lmb_alloc_base()
> API's") an additional allocation was mistakenly introduced resulting
> in ${kernel_comp_size} containing the address of a second 64mb region
> rather than the actual value of KERNEL_COMP_SIZE.
>
> Additionally, in b40d7b8f72f1 ("Merge patch series "lmb: use a single
> API for all allocations"") merge conflict resulted in an additional
> 128mb allocation for ${loadaddr} when CONFIG_FASTBOOT is enabled, where
> it should actually be set to the same value as ${fastboot_addr_r} to
> respect size constraints (and since it doesn't seem to interfer with
> any bootflows).
>
> Fixup both of these, freeing up 192mb of memory.
>
> Fixes: 6e4675b8e5d8 ("lmb: replace the lmb_alloc() and lmb_alloc_base()
> API's")
> Fixes: b40d7b8f72f1 ("Merge patch series "lmb: use a single API for all
> allocations"")
> Signed-off-by: Casey Connolly <casey.conno...@linaro.org>Whoops, thanks. In hind sight I should have cc'd you as well when merging all of that. -- Tom
signature.asc
Description: PGP signature
