diff mbox series

[U-Boot] configs: armada8k: use 2 DRAM banks

Message ID d302926e13718f32c57ff98ac767b3a3b30c5162.1542787172.git.baruch@tkos.co.il
State Accepted
Commit d3b2fb56c90038f5455016b29742d50706fa9dc5
Delegated to: Stefan Roese
Headers show
Series [U-Boot] configs: armada8k: use 2 DRAM banks | expand

Commit Message

Baruch Siach Nov. 21, 2018, 7:59 a.m. UTC
Commit 2b4d964718c0 ("arm64: mvebu: a8k: autodetect RAM size") added an
ATF query to get the detected RAM size on Armada 8K platforms. To be
usable we must have 2 DRAM banks. Set Armada 8K configurations to 2
banks.

Cc: Konstantin Porotchkin <kostap@marvell.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 configs/clearfog_gt_8k_defconfig      | 2 +-
 configs/mvebu_db_armada8k_defconfig   | 2 +-
 configs/mvebu_mcbin-88f8040_defconfig | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Comments

Stefan Roese Nov. 22, 2018, 8:38 a.m. UTC | #1
On 21.11.18 08:59, Baruch Siach wrote:
> Commit 2b4d964718c0 ("arm64: mvebu: a8k: autodetect RAM size") added an
> ATF query to get the detected RAM size on Armada 8K platforms. To be
> usable we must have 2 DRAM banks. Set Armada 8K configurations to 2
> banks.
> 
> Cc: Konstantin Porotchkin <kostap@marvell.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>   configs/clearfog_gt_8k_defconfig      | 2 +-
>   configs/mvebu_db_armada8k_defconfig   | 2 +-
>   configs/mvebu_mcbin-88f8040_defconfig | 2 +-
>   3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configs/clearfog_gt_8k_defconfig b/configs/clearfog_gt_8k_defconfig
> index 49b030766517..99f73efdb544 100644
> --- a/configs/clearfog_gt_8k_defconfig
> +++ b/configs/clearfog_gt_8k_defconfig
> @@ -10,7 +10,7 @@ CONFIG_SMBIOS_PRODUCT_NAME=""
>   CONFIG_DEBUG_UART=y
>   CONFIG_AHCI=y
>   CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_NR_DRAM_BANKS=1
> +CONFIG_NR_DRAM_BANKS=2
>   # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
> diff --git a/configs/mvebu_db_armada8k_defconfig b/configs/mvebu_db_armada8k_defconfig
> index cbdc130e98cd..bb29d2574d2e 100644
> --- a/configs/mvebu_db_armada8k_defconfig
> +++ b/configs/mvebu_db_armada8k_defconfig
> @@ -9,7 +9,7 @@ CONFIG_SMBIOS_PRODUCT_NAME=""
>   CONFIG_DEBUG_UART=y
>   CONFIG_AHCI=y
>   CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_NR_DRAM_BANKS=1
> +CONFIG_NR_DRAM_BANKS=2
>   # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
> diff --git a/configs/mvebu_mcbin-88f8040_defconfig b/configs/mvebu_mcbin-88f8040_defconfig
> index 9f72b76a3ff4..44436bc2a941 100644
> --- a/configs/mvebu_mcbin-88f8040_defconfig
> +++ b/configs/mvebu_mcbin-88f8040_defconfig
> @@ -9,7 +9,7 @@ CONFIG_SMBIOS_PRODUCT_NAME=""
>   CONFIG_DEBUG_UART=y
>   CONFIG_AHCI=y
>   CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_NR_DRAM_BANKS=1
> +CONFIG_NR_DRAM_BANKS=2
>   # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
> 

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan
Stefan Roese Nov. 30, 2018, 2:38 p.m. UTC | #2
On 21.11.18 08:59, Baruch Siach wrote:
> Commit 2b4d964718c0 ("arm64: mvebu: a8k: autodetect RAM size") added an
> ATF query to get the detected RAM size on Armada 8K platforms. To be
> usable we must have 2 DRAM banks. Set Armada 8K configurations to 2
> banks.
> 
> Cc: Konstantin Porotchkin <kostap@marvell.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Applied to u-boot-marvell/master.

Thanks,
Stefan
diff mbox series

Patch

diff --git a/configs/clearfog_gt_8k_defconfig b/configs/clearfog_gt_8k_defconfig
index 49b030766517..99f73efdb544 100644
--- a/configs/clearfog_gt_8k_defconfig
+++ b/configs/clearfog_gt_8k_defconfig
@@ -10,7 +10,7 @@  CONFIG_SMBIOS_PRODUCT_NAME=""
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 CONFIG_DISTRO_DEFAULTS=y
-CONFIG_NR_DRAM_BANKS=1
+CONFIG_NR_DRAM_BANKS=2
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 # CONFIG_DISPLAY_CPUINFO is not set
diff --git a/configs/mvebu_db_armada8k_defconfig b/configs/mvebu_db_armada8k_defconfig
index cbdc130e98cd..bb29d2574d2e 100644
--- a/configs/mvebu_db_armada8k_defconfig
+++ b/configs/mvebu_db_armada8k_defconfig
@@ -9,7 +9,7 @@  CONFIG_SMBIOS_PRODUCT_NAME=""
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 CONFIG_DISTRO_DEFAULTS=y
-CONFIG_NR_DRAM_BANKS=1
+CONFIG_NR_DRAM_BANKS=2
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 # CONFIG_DISPLAY_CPUINFO is not set
diff --git a/configs/mvebu_mcbin-88f8040_defconfig b/configs/mvebu_mcbin-88f8040_defconfig
index 9f72b76a3ff4..44436bc2a941 100644
--- a/configs/mvebu_mcbin-88f8040_defconfig
+++ b/configs/mvebu_mcbin-88f8040_defconfig
@@ -9,7 +9,7 @@  CONFIG_SMBIOS_PRODUCT_NAME=""
 CONFIG_DEBUG_UART=y
 CONFIG_AHCI=y
 CONFIG_DISTRO_DEFAULTS=y
-CONFIG_NR_DRAM_BANKS=1
+CONFIG_NR_DRAM_BANKS=2
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 # CONFIG_DISPLAY_CPUINFO is not set