Re: [PATCH 1/2] board_f: Add verbose DRAM layout print option

On 2026-08-02T21:33:37, Marek Vasut <[email protected]> wrote:
> board_f: Add verbose DRAM layout print option
>
> Make verbose DRAM layout print on boot configurable via newly introduced
> CONFIG_DISPLAY_DRAM_CONFIG_VERBOSE Kconfig option. This reuses existing
> DRAM layout debug print, but with additional rework to place the output
> at the end of DRAM size listing, print both bank start and end addresses,
> and skip empty banks. In case DEBUG is defined, the DRAM layout print is
> triggered unconditionally.
>
> The original debug print looked as follows, which puts the layout print
> between DRAM: size announcement (1) and duplicate DRAM: size print (2),
> does not skip empty banks (3), and does not print DRAM bank end (4).
>
> "
> Model: Renesas Ironhide board based on r8a78000
> DRAM:  <----------------------------------------------------------- (1)
> RAM Configuration:
> Bank #0: 1080000000 2 GiB
> Bank #1: 1200000000 4 GiB <---------------------------------------- (4)
> ...
> Bank #8: 40000000 2 GiB
> Bank #9: 0 0 Bytes <----------------------------------------------- (3)
> ...
> Bank #15: 0 0 Bytes
>
> DRAM:  2 GiB (total 32 GiB) <-------------------------------------- (2)
> Core:  68 devices, 26 uclasses, devicetree: separate
> "
>
> The new debug print and optionally verbose print looks as follows, the
> duplicate DRAM: prefix is removed (1), the layout is printed after the
> DRAM size print (3), the DRAM end address is printed (3) and there are
> no more empty banks printed (4):
>
> "
> Model: Renesas Ironhide board based on r8a78000
> DRAM:  2 GiB (total 32 GiB) <-------------------------------------- (1)
> RAM Configuration: <----------------------------------------------- (2)
> Bank #0: 0x1080000000 - 0x10ffffffff, 2 GiB <---------------------- (4)
> Bank #1: 0x1200000000 - 0x12ffffffff, 4 GiB
> Bank #2: 0x1400000000 - 0x14ffffffff, 4 GiB
> Bank #3: 0x1600000000 - 0x16ffffffff, 4 GiB
> Bank #4: 0x1800000000 - 0x18ffffffff, 4 GiB
> Bank #5: 0x1a00000000 - 0x1affffffff, 4 GiB
> Bank #6: 0x1c00000000 - 0x1cffffffff, 4 GiB
> Bank #7: 0x1e00000000 - 0x1effffffff, 4 GiB
> Bank #8: 0x40000000 - 0xbfffffff, 2 GiB <-------------------------- (3)
> Core:  68 devices, 26 uclasses, devicetree: separate
> "
>
> Signed-off-by: Marek Vasut <[email protected]>
> Reviewed-by: Ilias Apalodimas <[email protected]>
>
> common/Kconfig   |  6 ++++++
>  common/board_f.c | 33 ++++++++++++++++++++++++---------
>  2 files changed, 30 insertions(+), 9 deletions(-)

Reviewed-by: Simon Glass <[email protected]>