Dear Otavio Salvador, > On Thu, Jan 24, 2013 at 3:56 PM, Marek Vasut <ma...@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. > Regards, > > -- > Otavio Salvador O.S. Systems > E-mail: ota...@ossystems.com.br http://www.ossystems.com.br > Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot