From mboxrd@z Thu Jan  1 00:00:00 1970
From: Aneesh V <aneesh@ti.com>
Date: Wed, 15 Jun 2011 17:38:32 +0530
Subject: [U-Boot] [PATCH v2 08/22] omap: add spl support
In-Reply-To: <20110615120400.7523A655C6@gemini.denx.de>
References: <1298893591-17636-1-git-send-email-aneesh@ti.com>
	<1305472900-4004-9-git-send-email-aneesh@ti.com>
	<20110515195208.EA0971491B06@gemini.denx.de>
	<4DD13056.3000509@ti.com>
	<20110516154350.F16DB1491B07@gemini.denx.de>
	<4DD21CD8.2080409@ti.com>
	<20110517081637.69CE71491B09@gemini.denx.de>
	<4DDE5AFE.9000404@ti.com>
	<20110615101307.7DBFE82491@gemini.denx.de>
	<4DF88F45.9090905@ti.com>
	<20110615120400.7523A655C6@gemini.denx.de>
Message-ID: <4DF8A0C0.2040900@ti.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Wolfgang,

On Wednesday 15 June 2011 05:34 PM, Wolfgang Denk wrote:
> Dear Aneesh,
>
> In message<4DF88F45.9090905@ti.com>  you wrote:
>>
>>> I don't get this.  Why don't we just pass the required make target
>>> from the top level Makefile?  If we want to build "onenand-ipl-2k.bin"
>>> then this would result in running "make onenand-ipl-2k.bin" in the
>>> respective directory - this is then also independent of whether this
>>> image contains support for one specific boot device or for any set of
>>> different devices.
>>
>> Then you will have to deal with many such images in the top-level
>> Makefile(u-boot/Makefile). Instead why not have just one rule like the
>> below in the top-level Makefile for all the different spl's:
>>
>>
>> SPL    :$(TIMESTAMP_FILE) $(VERSION_FILE) depend tools
>>       $(MAKE) -C spl/board/$(BOARDDIR) all
>>
>> And leave the rest to the board level Makefiles
>
> We just discussed that there are cases where there may be no board
> level Makefile be needed at all, because there is no board specific
> code to handle.
>
> Also, why would there be many such images? We might just want to use
> more reasonable names.  We have "u-boot.bin", and this works fine for
> all boards, so why cannot we make it that "u-boot-nand.bin" works for
> all boards booting from NAND, and in general "u-boot-<DEV>.bin" works
> for all boards booting from a<DEV>  boot device?
>
> Instead of "onenand-ipl-2k.bin", we would just have a generic
> "u-boot-onenand.bin"
>
>> Of course, this is assuming the existing Makefile structure. With the
>> new Makefile structure you are suggesting this may not hold good.
>
> Why not?

I was saying that my suggestion of delegating everything to board level
Makefile will not work with the new top-down approach you are
suggesting in the other mail-chain.

best regards,
Aneesh