Re: [PATCH] lmb: Default to not-LMB_USE_MAX_REGIONS

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 873 bytes --]

On Wed, Jan 25, 2023 at 06:08:23PM -0500, Tom Rini wrote:

> The LMB code allows for picking a hard limit on the number of regions it
> can know of, or to dynamically allocate these regions. The reason for
> this choice is to allow for the compiler to perform a size optimization
> in the common case. This optimization however, is very small, ranging
> from 196 bytes to 15 bytes saved, or in some cases, being larger. Now
> that we also have more regions covered by LMB (in order to protect
> various parts of our self at run time), the default of 8 is also much
> easier to hit and leads to non-obvious error messages (which imply that
> an area is protected, not that we're out of areas to add to the list).
> 
> Switch to the dynamic use as the default.
> 
> Signed-off-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]