board: arbel: Limit the dram effective size to bank0 maximal size
authorJim Liu <jim.t90615@gmail.com>
Tue, 23 Apr 2024 07:22:10 +0000 (15:22 +0800)
committerTom Rini <trini@konsulko.com>
Fri, 3 May 2024 18:22:59 +0000 (12:22 -0600)
For 4GB dram size, the dram is divided into 2 banks and
the address space of these 2 banks are not concatenated.
Limit the gd->ram_top to not exceed bank0 top to prevent
accessing invalid memory region.

Signed-off-by: Jim Liu <JJLIU0@nuvoton.com>
board/nuvoton/arbel_evb/arbel_evb.c

index 8fc56c1..53c931c 100644 (file)
@@ -27,6 +27,15 @@ int board_init(void)
        return 0;
 }
 
+phys_size_t get_effective_memsize(void)
+{
+       /* Use bank0 only */
+       if (gd->ram_size > DRAM_2GB_SIZE)
+               return DRAM_2GB_SIZE;
+
+       return gd->ram_size;
+}
+
 int dram_init(void)
 {
        struct npcm_gcr *gcr = (struct npcm_gcr *)NPCM_GCR_BA;
@@ -70,21 +79,16 @@ int dram_init_banksize(void)
                gd->bd->bi_dram[1].start = DRAM_4GB_SIZE;
                gd->bd->bi_dram[1].size = DRAM_2GB_SIZE -
                        (DRAM_4GB_SIZE - DRAM_4GB_ECC_SIZE);
-               /* use bank0 only */
-               gd->ram_size = DRAM_2GB_SIZE;
                break;
        case DRAM_4GB_SIZE:
                gd->bd->bi_dram[0].size = DRAM_2GB_SIZE;
                gd->bd->bi_dram[1].start = DRAM_4GB_SIZE;
                gd->bd->bi_dram[1].size = DRAM_2GB_SIZE;
-               /* use bank0 only */
-               gd->ram_size = DRAM_2GB_SIZE;
                break;
        default:
                gd->bd->bi_dram[0].size = DRAM_1GB_SIZE;
                gd->bd->bi_dram[1].start = 0;
                gd->bd->bi_dram[1].size = 0;
-               gd->ram_size = DRAM_1GB_SIZE;
                break;
        }