[U-Boot] [PATCH 1/3] spl: add ymodem support

[Stefano Babic]

On 07/03/2012 20:30, Tom Rini wrote:
> On Wed, Mar 7, 2012 at 12:10 PM, Mikhail Kshevetskiy
> <mikhail.kshevetskiy at gmail.com> wrote:
>> On Wed, 7 Mar 2012 08:25:39 -0700
>> Tom Rini <trini at ti.com> wrote:
>>
>>> On Wed, Mar 07, 2012 at 12:54:05AM +0400, Mikhail Kshevetskiy wrote:
>>>
>>>> We have an omap l138 based board without jtag and empty spi flash.
>>>> UART is an only way to load something on this board, so we are using
>>>> uart to load spl image u-boot and then we are using ymodem to load
>>>> the rest part of u-boot.
>>>
>>> This conflicts a bit with the SPL ymodem support that's now in
>>> u-boot-arm/master and making its way into mainline.  Please rebase (and
>>> perhaps play some Makefile magic games to get
>>> arch/arm/cpu/armv7/omap-common/spl_ymodem.c available to davinci, and
>>> yes, we all know we need to unify more of the SPL codebase).
>>
>> Hm, spl_ymodem_load_image() should be arch independent, so I suggest moving it
>> and getcymodem() out from arch/arm/cpu/armv7/omap-common/spl_ymodem.c to
>> common/xyzModem.c (similarly to my patch).
>>
>> any comments, suggestions?
> 
> Stefano is working on trying to unify SPL, but the first step is
> getting the SPL loads Linux patch series in.

Yes - I have already tried to factorize SPL to make it common for all
SOCs, but I broke some boards and discussing with Tom we decided that it
is better to proceed step by step.

Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================