From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tim Harvey Date: Mon, 21 Apr 2014 11:28:57 -0700 Subject: [U-Boot] [PATCH 06/11] MX6: add struct for sharing data between SPL and uboot In-Reply-To: <53537CB5.9060307@compulab.co.il> 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: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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? Where is your code? I don't see any cm-fx6 in mainline or posted patchsets. Tim