[U-Boot] [PATCH v2 08/22] omap: add spl support

Aneesh V
2011-05-18T05:05:56Z

Hi Scott,

On Tuesday 17 May 2011 12:09 AM, Scott Wood wrote:
> On Mon, 16 May 2011 19:40:30 +0530
> Aneesh V<aneesh@ti.com> wrote:
>
>> Hi Wolfgang,
>>
>> On Monday 16 May 2011 01:22 AM, Wolfgang Denk wrote:
>>> I think, in the first step of this series, we should move the existing
>>> code from nand_spl and onenand_ipl into something like spl/nand and
>>> spl/onenand, respectivly, and make sure all teh existing systems still
>>> build. The we can add more such support.
>>
>> IMHO, we should just have spl/board/<vendor>/<board>/ .
>>
>> Let spl/board/<vendor>/<board>/Makefile decide what it wants to
>> build.
>
> That's what we do in nand_spl/onenand_spl -- the common files are more like
> libraries that the board makefile may or may not pull in (e.g. many
> Freescale boards use nand_boot_fsl_elbc.c or nand_boot_fsl_nfc.c instead of
> nand_boot.c).
>
> Renaming nand_spl seems like it should be the first step.
>
>> I mean something like this(similar changes for ONENAND):
>>
>> +.PHONEY : SPL
>
> .PHONY

Oops! I wonder how it was working then. My real patch too had the same
mistake!

best regards,
Aneesh