Hi Tim, On 04/21/14 21:28, Tim Harvey wrote: > On Sun, Apr 20, 2014 at 12:52 AM, Igor Grinberg <grinb...@compulab.co.il> > wrote: >> On 04/18/14 09:35, Tim Harvey wrote: >>> On Thu, Apr 17, 2014 at 4:44 AM, Stefano Babic <sba...@denx.de> 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. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
