public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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