[U-Boot] [PATCH 06/11] MX6: add struct for sharing data between SPL and uboot

Tim Harvey
2014-04-21T18:28:57Z

On Sun, Apr 20, 2014 at 12:52 AM, Igor Grinberg <grinberg@compulab.co.il> wrote:
> On 04/18/14 09:35, Tim Harvey wrote:
>> On Thu, Apr 17, 2014 at 4:44 AM, Stefano Babic <sbabic@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?
Where is your code? I don't see any cm-fx6 in mainline or posted
patchsets.

Tim