linux-mtd.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Roland Stigge <stigge@antcom.de>
To: dedekind1@gmail.com
Cc: Sebastian Huber <sebastian.huber@embedded-brains.de>,
	Linux MTD <linux-mtd@lists.infradead.org>
Subject: Re: ARM: LPC32XX SLC ECC Handling
Date: Wed, 29 Aug 2012 13:52:20 +0200	[thread overview]
Message-ID: <503E0274.7000205@antcom.de> (raw)
In-Reply-To: <1346229208.2848.441.camel@sauron.fi.intel.com>

On 08/29/2012 10:33 AM, Artem Bityutskiy wrote:
>>>> the LPC32XX SLC controller has a ECC feature that covers only
>>>> the main data area, but not the OOB data.  Is it the
>>>> responsibility of the user to protect its OOB data via an
>>>> ECC?  The LPC32XX MLC controller on the other hand covers the
>>>> complete OOB data with its ECC.  How can a user determine if
>>>> it has to protect its OOB data?
>>> 
>>> I guess it is good idea to CC the author of the drivers.
>> 
>> Thanks for CC'ing, would have missed it.
>> 
>> Yes, it's exactly as Sebastian described - only the MLC
>> controller covers OOB in ECC.
>> 
>> At this point, I need to give back the question to the MTD 
>> maintainers: Is there an API how we can help users at this point?
>> I would be happy to implement it.
> 
> Well, I guess from the user's POW the driver either protects OOB
> data with ECC or not. There is no interface for detecting this in
> MTD, but I do not see why someone could not create it.
> 
> Am I right that we are talking about the situation when OOB and
> data areas are covered by different ECCs, or this is about user
> writing data +oob at one go, and everything is covered by the same
> ECC?

In the above case of LPC32xx/MLC NAND (not sure if this is common with
other controllers), the controller covers 512 (sub)pages plus some OOB
bytes with a single ECC operation, storing it behind those covered OOB.

Roland

  reply	other threads:[~2012-08-29 11:52 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-08  9:16 ARM: LPC32XX SLC ECC Handling Sebastian Huber
2012-08-24 11:38 ` Artem Bityutskiy
2012-08-24 12:54   ` Roland Stigge
2012-08-29  8:33     ` Artem Bityutskiy
2012-08-29 11:52       ` Roland Stigge [this message]
2012-09-15  3:59         ` Brian Norris

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=503E0274.7000205@antcom.de \
    --to=stigge@antcom.de \
    --cc=dedekind1@gmail.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=sebastian.huber@embedded-brains.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;
as well as URLs for NNTP newsgroup(s).