[U-Boot] [RFC PATCH v2 03/28] arm: Only display frame buffer info if there is LCD/video support
Albert ARIBAUD
albert.u.boot at aribaud.net
Sat Jan 14 10:18:45 CET 2012
Hi Simon,
Le 07/01/2012 01:24, Simon Glass a écrit :
> This value has no meaning otherwise.
>
> Signed-off-by: Simon Glass<sjg at chromium.org>
> ---
>
> common/cmd_bdinfo.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c
> index ac8e78d..cdc595e 100644
> --- a/common/cmd_bdinfo.c
> +++ b/common/cmd_bdinfo.c
> @@ -369,7 +369,9 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> print_num("reloc off", gd->reloc_off);
> print_num("irq_sp", gd->irq_sp); /* irq stack pointer */
> print_num("sp start ", gd->start_addr_sp);
> +#if defined(CONFIG_LCD) || defined(CONFIG_VIDEO)
> print_num("FB base ", gd->fb_base);
> +#endif
> return 0;
> }
>
I understand the field can have a meaningful value only for those boards
with a display, but since it apparently exists on all boards, then it
should be displayed -- or, if you feel it should not exist for boards
without displays, then you should make its existence conditional as well.
Amicalement,
--
Albert.
More information about the U-Boot
mailing list