[PATCH 0/5] lmb: Add config for the number of memory and reserved regions

Tom Rini trini at konsulko.com
Tue Mar 2 21:50:53 CET 2021


On Tue, Feb 02, 2021 at 01:59:07PM +0100, Patrick Delaunay wrote:

> Hi,
> 
> I propose a update of the lmb library to allow the configuration
> with Kconfig of the number of memory and reserved regions in lmb
> libary:
> - CONFIG_LMB_RESERVED_REGIONS
> - CONFIG_LMB_MEMORY_REGIONS
> 
> By default, I keep the default value of 8 regions.
> 
> This serie avoids issue on stm32mp15 platform,
> as the kernel device tree defines many and non contiguous reserved
> regions.
> 
> 
> 
> Patrick Delaunay (5):
>   lmb: move CONFIG_LMB in Kconfig
>   lmb: remove lmb_region.size
>   lmb: Move lmb property arrays in struct lmb
>   lmb: Add 2 config to define the max number of regions
>   configs: stm32mp15: increase the number of reserved memory region in
>     lmb
> 
>  arch/arc/include/asm/config.h        |  2 --
>  arch/arm/include/asm/config.h        |  1 -
>  arch/m68k/include/asm/config.h       |  1 -
>  arch/microblaze/include/asm/config.h |  2 --
>  arch/mips/include/asm/config.h       |  1 -
>  arch/nds32/include/asm/config.h      |  1 -
>  arch/powerpc/include/asm/config.h    |  1 -
>  arch/riscv/include/asm/config.h      |  1 -
>  arch/sh/include/asm/config.h         |  2 --
>  arch/x86/include/asm/config.h        |  1 -
>  arch/xtensa/include/asm/config.h     |  2 --
>  configs/stm32mp15_basic_defconfig    |  1 +
>  configs/stm32mp15_trusted_defconfig  |  1 +
>  include/configs/10m50_devboard.h     |  5 ----
>  include/configs/3c120_devboard.h     |  5 ----
>  include/configs/sandbox.h            |  2 --
>  include/configs/x86-common.h         |  2 --
>  include/image.h                      |  2 +-
>  include/lmb.h                        | 34 ++++++++++++++++++++++++----
>  lib/Kconfig                          | 23 +++++++++++++++++++
>  lib/lmb.c                            | 14 ++++++------
>  scripts/config_whitelist.txt         |  1 -
>  22 files changed, 63 insertions(+), 42 deletions(-)

This series causes a little bit of size growth everywhere.  It's fatal
on nokia_rx51 (and caught via the test script).  We can bring nokia_rx51
back under size by disabling the optimized memcpy/memset functions, but
that's not ideal.  Can we do your changes here in a more size-concious
way?  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210302/7edb7d8c/attachment.sig>


More information about the U-Boot mailing list