[RFC PATCH 08/31] lmb: replcace the lmb_init_and_reserve() function

Tom Rini trini at konsulko.com
Mon Jun 10 19:31:02 CEST 2024


On Sat, Jun 08, 2024 at 12:22:17AM +0530, Sughosh Ganu wrote:
> With the changes to make the Logical Memory Block(LMB) allocations
> persistent and with the common memory regions being reserved during
> board init, the lmb_init_and_reserve() API can be removed and replaced
> with a lmb_add_memory() API, which adds all the available memory to
> the LMB pool.
> 
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
>  arch/arm/mach-apple/board.c          |  2 +-
>  arch/arm/mach-snapdragon/board.c     |  2 +-
>  arch/arm/mach-stm32mp/stm32mp1/cpu.c |  2 +-
>  cmd/bdinfo.c                         |  2 +-
>  cmd/load.c                           |  2 +-
>  fs/fs.c                              |  2 +-
>  include/lmb.h                        | 12 +++++++++++-
>  lib/lmb.c                            | 15 +++++++++++----
>  net/tftp.c                           |  2 +-
>  net/wget.c                           |  2 +-
>  test/cmd/bdinfo.c                    | 10 +---------
>  11 files changed, 31 insertions(+), 22 deletions(-)
> 
> diff --git a/arch/arm/mach-apple/board.c b/arch/arm/mach-apple/board.c
> index c877c7b94c..2e72d03edd 100644
> --- a/arch/arm/mach-apple/board.c
> +++ b/arch/arm/mach-apple/board.c
> @@ -776,7 +776,7 @@ int board_late_init(void)
>  {
>  	u32 status = 0;
>  
> -	lmb_init_and_reserve(gd->bd, (void *)gd->fdt_blob);
> +	lmb_add_memory(gd->bd);
>  
>  	/* somewhat based on the Linux Kernel boot requirements:
>  	 * align by 2M and maximal FDT size 2M

We already reserved gd->bd as part of the initr_lmb call. So I think
this commit needs rethinking, or am I missing something?

-- 
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/20240610/745ca3dc/attachment.sig>


More information about the U-Boot mailing list