From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mxs: mxsboot: Add support for SD card generation for i.MX23
Date: Thu, 24 Jan 2013 19:08:52 +0100 [thread overview]
Message-ID: <201301241908.52642.marex@denx.de> (raw)
In-Reply-To: <CAP9ODKoVJYa3HEFdbTAmxtNT1NMxN6QxNQ46pExHTTg_oTw-tQ@mail.gmail.com>
Dear Otavio Salvador,
> On Thu, Jan 24, 2013 at 3:56 PM, Marek Vasut <marex@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: otavio at ossystems.com.br http://www.ossystems.com.br
> Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
Best regards,
Marek Vasut
next prev parent reply other threads:[~2013-01-24 18:08 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-24 17:54 [U-Boot] [PATCH] mxs: mxsboot: Add support for SD card generation for i.MX23 Otavio Salvador
2013-01-24 17:56 ` Marek Vasut
2013-01-24 18:03 ` Otavio Salvador
2013-01-24 18:08 ` Marek Vasut [this message]
2013-01-24 18:31 ` Otavio Salvador
2013-01-24 18:39 ` Marek Vasut
2013-02-04 18:12 ` Fabio Estevam
2013-02-04 18:15 ` Otavio Salvador
2013-02-04 18:46 ` Marek Vasut
2013-02-04 18:50 ` Otavio Salvador
2013-02-04 20:58 ` Marek Vasut
2013-02-04 18:45 ` Marek Vasut
2013-01-25 12:35 ` Otavio Salvador
2013-01-25 12:40 ` Marek Vasut
2013-01-26 15:50 ` Otavio Salvador
2013-01-26 17:03 ` Marek Vasut
2013-01-26 17:19 ` Otavio Salvador
2013-01-26 17:46 ` Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201301241908.52642.marex@denx.de \
--to=marex@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox