[U-Boot] [PATCH] mxs: mxsboot: Add support for SD card generation for i.MX23

[Otavio Salvador]

On Thu, Jan 24, 2013 at 4:08 PM, Marek Vasut <marex at denx.de> wrote:
> Dear Otavio Salvador,
>
>> On Thu, Jan 24, 2013 at 3:56 PM, Marek Vasut <marex at denx.de> wrote:
>> > Dear Otavio Salvador,
>> >
>> >> The mxsboot now receives the SoC type as parameter to generate binary
>> >> compatible with the SoC. Currently the NAND support has not been add
>> >> for i.MX23 as it is not yet supported in U-Boot.
>> >
>> > Please fix the NAND support as well, then resubmit.
>>
>> I won't work on NAND now; first I wish to fix the NAND driver support
>> to later work in boot support.
>>
>> > The patch basically does dd if=u-boot.sb ... bs=512 seek=4 ; any kind of
>> > information can be stored in those first four blocks and the mx23 bootrom
>> > ignores it, so what's the gain of this?
>>
>> Well, it works fine for users. A good gain in my opinion.
>
> How is a simple documented dd if=... different? It's the same on imx, you have
> to dd u-boot.imx with some offset.
>
>> > I wonder, will MX28 bootrom ignore them as well? Then maybe we can get
>> > rid of all this SD-specific junk.
>>
>> Did not test but MX28 expects a BCB data structure (as said in 12.11.2
>> - MX28RM) while MX23 does not. So I think MX28 won't work without the
>> BCB.
>
> Fabio?
>
>> > Furthermore, I'd like to see all of this reworked as another plugin for
>> > mkimage.
>>
>> Yes; it would be a good long term solution but I don't want to hold it
>> due any of above reasons. It works fine so improvements can be done
>> later.
>
> We already have a solution:
>
> dd if=u-boot.sb of=/dev/sdX1 bs=512 seek=4
>
> I think this is enough for now, until all is fixed in proper sequence. That is,
> NAND driver and only after that, mxsboot for NAND _and_ SD . For now, let's hold
> off this patch, add the above dd stuff into documentation (doc/README.mx23) and
> then when all is ready, fix it all properly please.

Well; you blocked olinuxino patch until mxsboot where ported and the
dd with offset were not a solution for you, ... so ...

I prefer to have this as is and share documentation with mx28. The
NAND ought to be done providing same interface so one doc for it all.
I think change it in next version is wrong and confuse users.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br