[PATCH v4 6/8] configs: j7*_evm_r5_defconfig: Set NR_DRAM_BANKS to 2

[Santhosh Kumar K]

Hi, Bryan,

On 23/10/24 20:18, Bryan Brattlof wrote:
> On October 21, 2024 thus sayeth Santhosh Kumar K:
>> From: Neha Malcom Francis <n-francis at ti.com>
>>
>> Set CONFIG_NR_DRAM_BANKS to 2 as we have two banks described in the
>> memory/ node for lower and higher addressible DDR regions.
>>
>> This allows use of FDT functions from fdt_support.c to set up and fixup
>> the memory/ node correctly.
>>
>> Signed-off-by: Neha Malcom Francis <n-francis at ti.com>
>> ---
>>   configs/j7200_evm_r5_defconfig  | 1 +
>>   configs/j721e_evm_r5_defconfig  | 1 +
>>   configs/j721s2_evm_r5_defconfig | 1 +
>>   configs/j784s4_evm_r5_defconfig | 1 +
>>   4 files changed, 4 insertions(+)
>>
>> diff --git a/configs/j7200_evm_r5_defconfig b/configs/j7200_evm_r5_defconfig
>> index 774a9eff439d..78dbd44f845f 100644
>> --- a/configs/j7200_evm_r5_defconfig
>> +++ b/configs/j7200_evm_r5_defconfig
>> @@ -5,6 +5,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x70000
>>   CONFIG_SPL_GPIO=y
>>   CONFIG_SPL_LIBCOMMON_SUPPORT=y
>>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
>> +CONFIG_NR_DRAM_BANKS=2
> 
> Should these go into Kconfig somewhere? These boards will always have 2
> DRAAM banks unless someone is artificially limiting DDR sizes.

Thanks for the suggestion. We can make it default at Kconfig.

Regards,
Santhosh.

> 
> 
>>   CONFIG_SOC_K3_J721E=y
>>   CONFIG_K3_EARLY_CONS=y
>>   CONFIG_TARGET_J7200_R5_EVM=y
>> diff --git a/configs/j721e_evm_r5_defconfig b/configs/j721e_evm_r5_defconfig
>> index 3b4a7c3c0eeb..dc391f14b193 100644
>> --- a/configs/j721e_evm_r5_defconfig
>> +++ b/configs/j721e_evm_r5_defconfig
>> @@ -5,6 +5,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x70000
>>   CONFIG_SPL_GPIO=y
>>   CONFIG_SPL_LIBCOMMON_SUPPORT=y
>>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
>> +CONFIG_NR_DRAM_BANKS=2
>>   CONFIG_SOC_K3_J721E=y
>>   CONFIG_K3_EARLY_CONS=y
>>   CONFIG_K3_QOS=y
>> diff --git a/configs/j721s2_evm_r5_defconfig b/configs/j721s2_evm_r5_defconfig
>> index b6adb6a77d7e..374bc283be2d 100644
>> --- a/configs/j721s2_evm_r5_defconfig
>> +++ b/configs/j721s2_evm_r5_defconfig
>> @@ -5,6 +5,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x10000
>>   CONFIG_SPL_GPIO=y
>>   CONFIG_SPL_LIBCOMMON_SUPPORT=y
>>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
>> +CONFIG_NR_DRAM_BANKS=2
>>   CONFIG_SOC_K3_J721S2=y
>>   CONFIG_K3_EARLY_CONS=y
>>   CONFIG_K3_QOS=y
>> diff --git a/configs/j784s4_evm_r5_defconfig b/configs/j784s4_evm_r5_defconfig
>> index a11688194977..6fe36ab38f02 100644
>> --- a/configs/j784s4_evm_r5_defconfig
>> +++ b/configs/j784s4_evm_r5_defconfig
>> @@ -5,6 +5,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x10000
>>   CONFIG_SPL_GPIO=y
>>   CONFIG_SPL_LIBCOMMON_SUPPORT=y
>>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
>> +CONFIG_NR_DRAM_BANKS=2
>>   CONFIG_SOC_K3_J784S4=y
>>   CONFIG_K3_EARLY_CONS=y
>>   CONFIG_K3_QOS=y
>> -- 
>> 2.34.1
>>