From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 15 Jun 2015 10:17:52 -0400 Subject: [U-Boot] [PATCH] keystone2: use detected ddr3a size In-Reply-To: <1434372481-1166-1-git-send-email-vitalya@ti.com> References: <1434372481-1166-1-git-send-email-vitalya@ti.com> Message-ID: <20150615141752.GP1728@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Jun 15, 2015 at 08:48:01AM -0400, Vitaly Andrianov wrote: > KS2 u-boot detects the ddr3a size installed to EVM. The detected size can > be used instead of environment variable. Because the ddr3 configuration is > done before relocation we cannot use a global variable to pass the > ddr3_size to ft_board_setup(). Instead we have to use the global data > structure. > > Because KS2 u-boot works in 32 bit address space the existing ram_size > global data filed cannot be used. The maximum, which the get_ram_size() > can detect is 2GB only. This patch creates the ddr3_size filed in the > arch_global_data structure, which is used for that purpose. > > Signed-off-by: Vitaly Andrianov So we've got a few possibilities here, yes? Since we have the ability to change the DDR modules on the board and read the sizes in the SPD information U-Boot is the place where the board can find out if we have say 1GB or 2GB of memory and thus has to be the one to correctly populate the device tree. So the "fix" that we're talking about for Calxeda can't be applied here. But this also brings up http://patchwork.ozlabs.org/patch/281094/ (and the follow-up of http://patchwork.ozlabs.org/patch/291219/ and http://patchwork.ozlabs.org/patch/291247/) where no, we have a problem that we need to fix. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: