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