[U-Boot] [PATCH V4 2/2] mcx: support for HTKW mcx board

[Tom Rini]

On Tue, Dec 13, 2011 at 5:06 PM, Ilya Yanok <yanok at emcraft.com> wrote:
> Hi Tom,
>
> On 14.12.2011 01:47, Tom Rini wrote:
>>> As for the devkit8000 boot problem, I have BSS set up close to the top
>>> of the RAM. Probably that's the reason I don't see any problems.
>>>
>>> Unfortunately, I can't recall why I had moved BSS... It clashed with
>>> something but I don't remember the details.
>>
>> OK.  I _want_ to put this into my /next branch (and thanks for working
>> over all of the issues that've popped up since you first posted the
>> series) but I also would like to try and keep differences for
>> difference sake out of the SPL implementations since I know other
>> folks are starting to look at the code and port their custom boards or
>> in-tree boards and I'm sure they'll go "why does X place things here
>> and Y there and my board isn't working?".  So if you can try and
>> recall what the clash was (and we can comment it, or maybe it was the
>> FAT thing :)) I'd really appreciate it.
>
> I see. I've looked through my old commit histories but haven't found
> anything interesting. All I can say that it wasn't FAT cause I've
> changed BSS location before I've even started playing with MMC SPL.
>
> Probably I just didn't want to change TEXT_BASE... Ah, now I recall.
> I tried to make SPL working without breaking the board, so I loaded SPL
> to SRAM with BDI and tried to make it work with the u-boot.bin I had in
> NAND. That's why I had to preserve TEXT_BASE and move BSS instead.
> So that seems unrelated to the problem you observe on devkit8000...

Ah-ha!  Whenever I hit that point I always went "oh, right, I forgot
to adjust CONFIG_SYS_TEXT_BASE" :)  So, would you mind either adding a
comment that says in effect what you've got here "Keep at ... so that
we can still be loaded by non-SPL loaders" or whack everything around
and match the am3517 crane/evm and the omap3 boards?  Thanks!

-- 
Tom