public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Bo Shen <voice.shen@atmel.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 0/2] Add atmel ROM code image
Date: Thu, 24 Apr 2014 11:03:39 +0800	[thread overview]
Message-ID: <53587F0B.9080006@atmel.com> (raw)
In-Reply-To: <1398263365-17931-1-git-send-email-andreas.devel@googlemail.com>

Hi Andreas,
   Thanks for your work.

On 04/23/2014 10:29 PM, Andreas Bie?mann wrote:
> This series add atmelimage support to mkimage.
>
> An atmelimage is a quite dumb image type cause it has no real header. The file
> is mostly unmodified but the 6'th ARM vector gets replaced by the image size
> to load.
>
> Heiko, I know your approach setting the vector in start.S but I think this
> solution is a bit smarter. We would need to patch at least 2 start.S files
> (arm926ejs and armv7), maybe also the arm920t one too. In fact is the
> conversion of executable BLOB to ROM detected executable BLOB something that
> should really be done afterwards as all other SoC vendors do.
>
>
> Andreas Bie?mann (2):
>    mkimage: add atmelimage
>    arm:at91: enable ROM loadable atmel image
>
>   arch/arm/cpu/armv7/at91/config.mk |   10 +++++
>   common/image.c                    |    1 +
>   include/image.h                   |    1 +
>   spl/Makefile                      |    5 +++
>   tools/Makefile                    |    1 +
>   tools/atmelimage.c                |   88 +++++++++++++++++++++++++++++++++++++
>   tools/imagetool.c                 |    2 +
>   tools/imagetool.h                 |    1 +
>   8 files changed, 109 insertions(+)
>   create mode 100644 arch/arm/cpu/armv7/at91/config.mk
>   create mode 100644 tools/atmelimage.c
>

For this patch set, test ok on sama5d33ek with spi flash and mmc boot.

For nand flash boot, it doesn't support to add PMECC header, so only 
works in software ECC or none ECC mode (that means, u-boot flash 
boot.bin to NAND flash won't work). So, do you plan to add following patch?

Best Regards,
Bo Shen

      parent reply	other threads:[~2014-04-24  3:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-23 14:29 [U-Boot] [RFC PATCH 0/2] Add atmel ROM code image Andreas Bießmann
2014-04-23 14:29 ` [U-Boot] [RFC PATCH 1/2] mkimage: add atmelimage Andreas Bießmann
2014-04-24  3:09   ` Bo Shen
2014-04-24  5:23     ` Andreas Bießmann
2014-04-23 14:29 ` [U-Boot] [RFC PATCH 2/2] arm:at91: enable ROM loadable atmel image Andreas Bießmann
2014-04-24  3:10   ` Bo Shen
2014-04-24  3:03 ` Bo Shen [this message]

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=53587F0B.9080006@atmel.com \
    --to=voice.shen@atmel.com \
    --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