[U-Boot] [PATCH 5/7] Tegra114: Dalmore: Add DT files

[Stephen Warren]

On 01/17/2013 03:58 PM, Tom Warren wrote:
> Stephen,
> 
> On Thu, Jan 17, 2013 at 3:25 PM, Stephen Warren <swarren at wwwdotorg.org> wrote:
>> On 01/17/2013 10:58 AM, Tom Warren wrote:
>>> Stephen,
>>>
>>> On Wed, Jan 16, 2013 at 3:47 PM, Stephen Warren <swarren at wwwdotorg.org> wrote:
>>>> On 01/16/2013 02:14 PM, Tom Warren wrote:
>>>>> These are stripped down for bringup, They'll be filled out later
>>>>> to match-up with the kernel DT contents, and/or as devices are
>>>>> brought up (mmc, usb, spi, etc.).
>>>>
>>>>> diff --git a/board/nvidia/dts/tegra114-dalmore.dts b/board/nvidia/dts/tegra114-dalmore.dts
>>>>
>>>>> +     memory {
>>>>> +             device_type = "memory";
>>>>> +             reg = <0x80000000 0x40000000>;
>>>>> +     };
>>>>
>>>> Indeed, that says just 1GB...
>>>>
>>>
>>> True, but that's what's in current Cardhu DTS, and Harmony, Seaboard,
>>> and Ventana DTS files. Only Whistler has 0x20000000 (512MB).
>>>
>>> How is this used, and how do we keep it in sync with what's really on
>>> the board/in the ODMDATA field?
>>
>> I don't believe U-Boot uses that property from the DT. Probably the DT
>> isn't parsed early enough for it to be useful. But if we add it to the
>> .dts file, it may as well be correct even so. Or, perhaps we should just
>> remove it from the U-Boot DT since it isn't useful.
> 
> Another candidate for a clean-up pass patchset over all Tegra code.
> For now, I'll bump it to 2GB for Dalmore (and Cardhu), since that's
> what's on the sample boards.

1GB would be safer for Cardhu; there are certainly very commonly used
Cardhu SKUs with only 1GB of RAM. I have one. In fact, I once heard that
most Linux-oriented Cardhu were 1GB and the 2GB devices were mostly used
for Windows development.