[RFC PATCH 28/31] cmd: bdinfo: only dump the current LMB memory

Heinrich Schuchardt xypron.glpk at gmx.de
Sat Jun 8 05:59:37 CEST 2024



Am 7. Juni 2024 20:52:37 MESZ schrieb Sughosh Ganu <sughosh.ganu at linaro.org>:
>The LMB memory map is now persistent and global. There is therefore no
>need to add memory to the LMB memory map as part of the bdinfo
>command. Only dump the current available and reserved memory as part
>of the bdinfo command.

Would it make sense to move printing lmb memory to a separate command instead?

Best regards

Heinrich

>
>Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
>---
> cmd/bdinfo.c | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c
>index fc408e9820..0f343203ef 100644
>--- a/cmd/bdinfo.c
>+++ b/cmd/bdinfo.c
>@@ -163,7 +163,6 @@ static int bdinfo_print_all(struct bd_info *bd)
> 	bdinfo_print_num_l("multi_dtb_fit", (ulong)gd->multi_dtb_fit);
> #endif
> 	if (IS_ENABLED(CONFIG_LMB) && gd->fdt_blob) {
>-		lmb_add_memory(gd->bd);
> 		lmb_dump_all_force();
> 		if (IS_ENABLED(CONFIG_OF_REAL))
> 			printf("devicetree  = %s\n", fdtdec_get_srcname());


More information about the U-Boot mailing list