[U-Boot] [PATCH V2 6/6] arm: mx5: Add support for DENX M53EVK

[Stefano Babic]

On 24/04/2013 20:52, Marek Vasut wrote:
>> But you forget that a single image can be saved on multiple storage:
>> u-boot.imx can be stored on SD or NOR or SPI-NOR, and that is the reason
>> for having SOC-specific extension.
>>
>> I agree with Benoit: at the moment, only people working with i.MX know
>> that u-boot.im runs on SD. The third solution proposed by Benoit has the
>> drawback that probably not all boards need u-boot.imx (a board without
>> SD for example). At least we need an update of the README, but I think
>> it is not bad to have a new entry in boards.cfg.
>>
>> Apart of that and not related to this patch, if we in future use SPL
>> also for booting from SD, we can get a single way to boot from different
>> storage. TI based SOCs already do this: same SPL, it checks from SD and
>> NAND.
> 
> Ok, I fail to grasp what is wanted from me. Shall I rework the patch somehow? 
> How? Do we want m53evk_sd and m53evk_nand targets ?

At least add documentation in the README. It is only your choice if you
will add two different targets to boards.cfg

Best regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================