All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.