From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Sun, 21 Apr 2013 22:29:02 +0200
Subject: [U-Boot] [PATCH V2 6/6] arm: mx5: Add support for DENX M53EVK
In-Reply-To: <1643889946.6492.1366573601954.JavaMail.root@advansee.com>
References: <1366559547-9063-1-git-send-email-marex@denx.de>
	<201304212112.31672.marex@denx.de>
	<1643889946.6492.1366573601954.JavaMail.root@advansee.com>
Message-ID: <201304212229.03069.marex@denx.de>
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 Beno?t Th?baudeau,

> Dear Marek Vasut,
> 
> On Sunday, April 21, 2013 9:12:31 PM, Marek Vasut wrote:
> > Dear Beno?t Th?baudeau,
> > 
> > > Dear Marek Vasut,
> > > 
> > > On Sunday, April 21, 2013 5:52:27 PM, Marek Vasut wrote:
> > > > Add basic support for the DENX M53EVK board. Currently supported is:
> > > > MMC (incl. booting)
> > > > 
> > >              ^
> > > 
> > > Can you clarify this, please?
> > 
> > Use u-boot.imx for SD booting as usual.
> > 
> > > spl_boot_device() points only to NAND, so
> > > you're clearly talking about hardware MMC boot, and not about hardware
> > > NAND boot followed by SPL payload fetched from MMC. But MMC boot does
> > > not need SPL here, in which case you will have to generate a simple
> > > u-boot.imx, or you will rather want to use u-boot-with-spl.imx for SD
> > > (NAND header dropped to leave room for MBR). And in the latter case,
> > > why have spl_boot_device() point to NAND for MMC boot?
> > 
> > No, regular u-boot.imx will be used for SD boot.
> 
> OK. So this will require to call make with u-boot.imx as the explicit
> target. Should this be documented somewhere, perhaps in a README file for
> this board?
> 
> Another solution would be, like for woodburn, to have an sd-specific
> config: - m53evk_nand_config would define CONFIG_SPL from boards.cfg, so
>    u-boot-with-nand-spl.imx would be generated.
>  - mx53evk_sd_config would not define CONFIG_SPL from boards.cfg, so
> u-boot.imx would be generated.
> And CONFIG_SPL would be removed from m53evk.h.
> 
> Or, change the various config.mk in order to build u-boot.imx even if
> CONFIG_SPL is defined, which would be useless for some boards, but useful
> here in order to avoid having 2 configs for almost the same build, while
> still not having to explicitly give a make target.

I'd love to see generic u-boot.nand , u-boot.sd etc. targets instead of these 
CPU specific stuffs.

Best regards,
Marek Vasut