From mboxrd@z Thu Jan  1 00:00:00 1970
From: Vipin Kumar <vipin.kumar@st.com>
Date: Fri, 7 Dec 2012 15:17:26 +0530
Subject: [U-Boot] [PATCH resend 1/2] u-boot/spl: Add u-boot-spl.img to
	u-boot targets
In-Reply-To: <50C08AA5.2010006@denx.de>
References: <0914ca07f3fa7549188e349e7a0a32c3d6521321.1354786138.git.vipin.kumar@st.com>
	<50C08AA5.2010006@denx.de>
Message-ID: <50C1BB2E.7090806@st.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

On 12/6/2012 5:38 PM, Stefan Roese wrote:
> On 12/06/2012 10:29 AM, Vipin Kumar wrote:
>> Signed-off-by: Vipin Kumar<vipin.kumar@st.com>
>
> Is this new build target really needed? Please take a look at the
> recently added/renamed targets (e.g. u-boot-with-spl.bin or u-boot-img.bin).
>

Hmm, this is an mkimage version of spl binary. This mkimage header is 
detected by the BootROM in spear devices for loading and booting the SPL 
binary

> If this does not fit, then please add a short description about this new
> build target and which SoC's need it and why.
>

You mean add a short description in commit log or in README ?

Vipin

> Thanks,
> STefan
>
>