From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Grinberg Date: Tue, 22 Apr 2014 10:44:48 +0300 Subject: [U-Boot] [PATCH 06/11] MX6: add struct for sharing data between SPL and uboot In-Reply-To: References: <1396504871-1454-1-git-send-email-tharvey@gateworks.com> <1396504871-1454-7-git-send-email-tharvey@gateworks.com> <534BD60F.3030504@denx.de> <534F9F1B.5080209@denx.de> <534FB95B.3030206@compulab.co.il> <534FBEB3.3030909@denx.de> <53537CB5.9060307@compulab.co.il> Message-ID: <53561DF0.7020300@compulab.co.il> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tim, On 04/21/14 21:28, Tim Harvey wrote: > On Sun, Apr 20, 2014 at 12:52 AM, Igor Grinberg wrote: >> On 04/18/14 09:35, Tim Harvey wrote: >>> On Thu, Apr 17, 2014 at 4:44 AM, Stefano Babic wrote: >>>> Hi Igor, hi Tim >>>> >>>> On 17/04/2014 13:22, Igor Grinberg wrote: >>>> >>>>> >>>>> get_ram_size() works on cm-fx6 all DRAM configurations. >>>> >>>> As on most boards in mainline ;-) >>>> >>> >>> It looks like I mis-interpreted the failure. This issue is that >>> get_ram_size() only works with powers of 2 and maxes out at 2GB due to >>> 32bit addressing. I was simply passing in a bogus value. I'm not sure >>> how you would deal with detecting systems with 4GB but I don't need to >>> support that for my boards currently. >>> >> >> cm-fx6 supports 4GB and detects it properly. > > Igor, > > That's interesting. What value are you passing get_ram_size() then? Well, we do it in several get_ram_size() calls and decide according to returned results. > Where is your code? I don't see any cm-fx6 in mainline or posted > patchsets. Right, it is work in progress. Currently, it is based on 2014.01. Once it is ready, we re-base and post it. -- Regards, Igor.