On 07/04/2025 18:59, Stephan Gerhold wrote:
DB410c has exactly 1 GiB of RAM. Some of it is reserved, but this is
described separately in the DT.

This was fixed before in commit 1d667227ea51 ("board: dragonboard410c: Fix
PHYS_SDRAM_1_SIZE"), but was reintroduced when DB410c was converted to use
the upstream device tree.

Note that there are variants of apq8016-sbc with 2 GiB RAM (e.g. the
Geniatech DB4). They need the WIP SMEM memory map parsing [1] to use the
full amount of RAM.

[1]: 
https://lore.kernel.org/u-boot/20241124-b4-modernise-smem-v1-0-b7852c11b...@linaro.org/T/

Fixes: ed8fbd2889fc ("dts: msm8916: replace with upstream DTS")
Signed-off-by: Stephan Gerhold <stephan.gerh...@linaro.org>
---
  arch/arm/dts/apq8016-sbc-u-boot.dtsi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/apq8016-sbc-u-boot.dtsi 
b/arch/arm/dts/apq8016-sbc-u-boot.dtsi
index 
585d54d29623823145e6b43eda89c05f31fc4403..c8a46ed14483f6939e7a666b2b279c803476ac63
 100644
--- a/arch/arm/dts/apq8016-sbc-u-boot.dtsi
+++ b/arch/arm/dts/apq8016-sbc-u-boot.dtsi
@@ -6,7 +6,7 @@
  / {
        /* When running as a first-stage bootloader this isn't filled in 
automatically */
        memory@80000000 {
-               reg = <0 0x80000000 0 0x3da00000>;
+               reg = <0 0x80000000 0 0x40000000>;
        };
  };

Reviewed-by: Neil Armstrong <neil.armstr...@linaro.org>

Reply via email to