All of lore.kernel.org
 help / color / mirror / Atom feed
From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 24/28] armv8/ls2085aqds: NAND boot support
Date: Fri, 20 Mar 2015 14:23:35 -0700	[thread overview]
Message-ID: <550C8FD7.5080208@freescale.com> (raw)
In-Reply-To: <1426886140.27998.206.camel@freescale.com>



On 03/20/2015 02:15 PM, Scott Wood wrote:
> On Fri, 2015-03-20 at 12:21 -0700, York Sun wrote:
>> From: Scott Wood <scottwood@freescale.com>
>>
>> This adds NAND boot support for LS2085AQDS, using SPL framework.
>>
>> To form a NAND image, append u-boot-with-spl.bin after a proper
>> nand boot RCW and flash to the beginning of NAND.
> 
> Do we want to do it this way, or should we keep the RCW in a separate
> block?

I would like to see RCW in a separated block.
> 
> What constitutes a "proper nand boot RCW" (those were not my words)?
> There are details in this patch regarding offsets that need to match
> details in the PBI (which is more than just RCW).

It is not your original words. Yours was "To form a NAND image, append
u-boot-with-spl.bin after PBL_0x3_0x07_1333_nand.bin and flash to the beginning
of NAND.". I try to make the message generic. A proper nand boot RCW means the
RCW should contains PBI commands to set bootloc and block copy the image. Since
RCW is not in the scope of u-boot, I cannot refer to any specific file.

Do you want to include an RCW file into u-boot, like those for powerpc boards?
> 
> It would also be nice to sort out loading the environment during SPL.
That can be an additional patch when you have it.

York

  reply	other threads:[~2015-03-20 21:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-20 19:21 [U-Boot] [PATCH v4 04/28] armv8/ls2085a: Fix generic timer clock source York Sun
2015-03-20 19:21 ` [U-Boot] [PATCH v4 24/28] armv8/ls2085aqds: NAND boot support York Sun
2015-03-20 21:15   ` Scott Wood
2015-03-20 21:23     ` York Sun [this message]
2015-03-20 21:33       ` Scott Wood
2015-03-20 21:44         ` York Sun
2015-03-21 23:47           ` Scott Wood
2015-03-20 21:37     ` Tom Rini
2015-03-20 21:44       ` Scott Wood
2015-03-21  0:18         ` Tom Rini
2015-03-21  0:28           ` Scott Wood
2015-03-23  0:46             ` Tom Rini
2015-03-20 19:21 ` [U-Boot] [PATCH v4 26/28] armv8/ls2085ardb: Enable NAND SPL support York Sun

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=550C8FD7.5080208@freescale.com \
    --to=yorksun@freescale.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 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.