[U-Boot] [PATCH] rpi4: enable dram bank initialization

[Matthias Brugger]

On 06/09/2019 14:11, Alexander Graf wrote:
> 
> On 06.09.19 13:56, matthias.bgg at kernel.org wrote:
>> From: Matthias Brugger <mbrugger at suse.com>
>>
>> When booting through the efi stub, the memory map get's created by
>> reading the dram bank information. Depending on the version of the RPi4
>> this information changes. Read the device tree to initialize the dram
>> bank data structure. This way the kernel is able to access the whole
>> range of available memory.
>>
>> Signed-off-by: Matthias Brugger <mbrugger at suse.com>
>> ---
>> This patch is based on basic RPi4 support implemented by series:
>> https://www.mail-archive.com/u-boot@lists.denx.de/msg335667.html
>>
>> To actually work correctly we need the series that fixes the libftd:
>> https://patchwork.ozlabs.org/cover/1158304/
>>
>>   board/raspberrypi/rpi/rpi.c | 8 ++++++++
>>   configs/rpi_4_defconfig     | 2 +-
>>   2 files changed, 9 insertions(+), 1 deletion(-)
>>
>> diff --git a/board/raspberrypi/rpi/rpi.c b/board/raspberrypi/rpi/rpi.c
>> index fa57d50c95..eea8a69551 100644
>> --- a/board/raspberrypi/rpi/rpi.c
>> +++ b/board/raspberrypi/rpi/rpi.c
>> @@ -312,6 +312,14 @@ int dram_init(void)
>>       return 0;
>>   }
>>   +#ifdef CONFIG_BCM2711
>> +int dram_init_banksize(void)
>> +{
>> +    return fdtdec_decode_ram_size(gd->fdt_blob, NULL, 0, NULL,
> 
> 
> This also depends on CONFIG_OF_BOARD, no?
> 

I would need to double check if at this point gd->fdt_blob is in it's final
state or might get updated afterwards.

Actually I think we should change all RPi configs to OF_BOARD, which would also
be necessary to implement a single binary for RPi3 and RPi4. But that's another
story.