On Fri, Jun 12, 2020 at 03:41:18PM +0300, Tero Kristo wrote:

> Dump lmb status from the bdinfo command. This is useful for seeing the
> reserved memory regions from the u-boot cmdline.
> 
> Signed-off-by: Tero Kristo <[email protected]>
> ---
>  cmd/bdinfo.c  |  8 +++++++-
>  include/lmb.h |  1 +
>  lib/lmb.c     | 42 +++++++++++++++++++++++-------------------
>  3 files changed, 31 insertions(+), 20 deletions(-)
> 
> diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c
> index dba552b03f..05e1b27de0 100644
> --- a/cmd/bdinfo.c
> +++ b/cmd/bdinfo.c
> @@ -10,6 +10,7 @@
>  #include <common.h>
>  #include <command.h>
>  #include <env.h>
> +#include <lmb.h>
>  #include <net.h>
>  #include <vsprintf.h>
>  #include <asm/cache.h>
> @@ -365,8 +366,13 @@ static int do_bdinfo(struct cmd_tbl *cmdtp, int flag, 
> int argc,
>  #if CONFIG_IS_ENABLED(MULTI_DTB_FIT)
>       print_num("multi_dtb_fit", (ulong)gd->multi_dtb_fit);
>  #endif
> -     if (gd->fdt_blob)
> +     if (gd->fdt_blob) {
> +             struct lmb lmb;
>               print_num("fdt_blob", (ulong)gd->fdt_blob);
> +             lmb_init_and_reserve(&lmb, gd->bd, (void *)gd->fdt_blob);
> +             lmb_dump_all_force(&lmb);
> +     }
> +
>       print_cpu_word_size();
>  
>       return 0;

Can you please rebase this on top of master?  Simon's series to cleanup
bdinfo stuff has been applied, thanks.  The rest of your changes are
fine and under test now.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to