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

Igor Grinberg
2014-04-22T07:44:48Z

Hi Tim,

On 04/21/14 21:28, Tim Harvey wrote:
> 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?

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.