All of lore.kernel.org
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Status of fsl_elbc_nand driver and 4k page NAND / 4bit ECC
Date: Tue, 15 May 2012 17:47:03 -0500	[thread overview]
Message-ID: <4FB2DCE7.9010908@freescale.com> (raw)
In-Reply-To: <CABXAApF=yd1c80XFZ45wvieFSM+4CT07dBgJebWtkLtcndFsqg@mail.gmail.com>

On 05/08/2012 03:07 PM, Rafael Beims wrote:
> Based on your response, would it be OK to implement the marker in the
> bbt as something like Liu Shuo suggested ('M', 'B', 'b', 't', 0) and
> then do the scanning as follows:

I'd leave the BBT marker alone, and have the migration marker be located
elsewhere in the OOB that doesn't conflict with a BBT.  That way the BBT
code is untouched and the marker scanner only needs to consider one kind
of marker.

> Scan the bbt checking for the migrated flag as described. If we just
> find the default bbt marker, continue searching in the next block
> downwards to verify if we can find another marker indicating the
> migration (and completely independent of the bbt in this case).

Basically yes, except decouple it from the BBT.  The BBT code works as
always, and the migration marker code scans as described above.  If they
happen to be in the same block, fine -- the scanning code doesn't care
either way; only the marking code needs to know which to do.

> This second marker in this case would have to be written in a new
> block, or would it be OK to write it in a new page?
> It seems to me that a new block would be more appropriate, because you
> mentioned that a developer would like to erase the bbt sometimes, and
> that would imply the erasure of the entire block.

Right.

> What are your thought's about this suggestion?
> What would be an appropriate marker for the second block? It seems
> that the usual (at least for the bbt) is a marker with a character
> sequence. Should I just create a different character sequence and
> write it in the oob of the first page of the next block to indicate
> "developer" migration?

I tend to prefer random numbers over short strings for magic numbers, to
reduce the odds of collision, but whichever.  Just pick an offset into
the OOB that doesn't conflict with ECC or the BBT marker.

-Scott

      reply	other threads:[~2012-05-15 22:47 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CABXAApH0ffGMbC1DKPm3GGZyNO=cfzAOWmfN3VtWuUPXGwRY+Q@mail.gmail.com>
2012-04-11 21:28 ` [U-Boot] Status of fsl_elbc_nand driver and 4k page NAND / 4bit ECC Rafael Beims
2012-04-11 21:54   ` Scott Wood
     [not found]     ` <CABXAApFcrOhw0pF-w3enVqu8vY9xsEaViUpP1YujCDQeRcMRbQ@mail.gmail.com>
     [not found]       ` <4F862049.3040509@freescale.com>
2012-04-13 22:15         ` Rafael Beims
2012-04-16 21:10           ` Scott Wood
2012-05-08 20:07             ` Rafael Beims
2012-05-15 22:47               ` Scott Wood [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=4FB2DCE7.9010908@freescale.com \
    --to=scottwood@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.