[PATCH 27/40] spl: sandbox: initialise the ram banksize in spl

Simon Glass sjg at chromium.org
Fri Jul 26 01:32:34 CEST 2024


Hi Sughosh,

On Wed, 24 Jul 2024 at 00:05, Sughosh Ganu <sughosh.ganu at linaro.org> wrote:
>
> Initialise the ram bank information for sandbox in SPL. This is needed
> for initialising the LMB memory map as part of the platform init.

I don't understand what is going on here...you are moving code into a
function and should explain why.

>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
> Changes since rfc: New patch
>
>  arch/sandbox/cpu/spl.c | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/arch/sandbox/cpu/spl.c b/arch/sandbox/cpu/spl.c
> index 9ad9da686c..22cd916685 100644
> --- a/arch/sandbox/cpu/spl.c
> +++ b/arch/sandbox/cpu/spl.c
> @@ -121,6 +121,15 @@ static int load_from_image(struct spl_image_info *spl_image,
>  }
>  SPL_LOAD_IMAGE_METHOD("sandbox_image", 7, BOOT_DEVICE_BOARD, load_from_image);
>
> +int dram_init_banksize(void)
> +{
> +       /* These are necessary so TFTP can use LMBs to check its load address */
> +       gd->bd->bi_dram[0].start = gd->ram_base;
> +       gd->bd->bi_dram[0].size = get_effective_memsize();
> +
> +       return 0;
> +}
> +
>  void spl_board_init(void)
>  {
>         struct sandbox_state *state = state_get_current();
> @@ -128,10 +137,6 @@ void spl_board_init(void)
>         if (!CONFIG_IS_ENABLED(UNIT_TEST))
>                 return;
>
> -       /* These are necessary so TFTP can use LMBs to check its load address */
> -       gd->bd->bi_dram[0].start = gd->ram_base;
> -       gd->bd->bi_dram[0].size = get_effective_memsize();
> -
>         if (state->run_unittests) {
>                 struct unit_test *tests = UNIT_TEST_ALL_START();
>                 const int count = UNIT_TEST_ALL_COUNT();
> --
> 2.34.1
>

Regards,
Simon


More information about the U-Boot mailing list