linux-mtd.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Angus CLARK <angus.clark@st.com>
To: linux-mtd@lists.infradead.org
Subject: Re: Changing the ECC method for a running system (UBI seems to use OOB?)
Date: Fri, 12 Apr 2013 11:37:35 +0100	[thread overview]
Message-ID: <5167E3EF.9020505@st.com> (raw)
In-Reply-To: <5167D00A.1090006@ammonit.com>

Just to add one other point, you may need to skip programming "empty" pages when
transcoding the data to the new ECC format.  This will ensure the "empty" pages
can be programmed correctly, when UBIFS attempts to use them.

Cheers,

Angus

On 04/12/2013 10:12 AM, Steffen Kühn wrote:
> Hello David,
> 
>> We had exactly the same issue with our embedded systems (I'm really mad at
>> Micron that they released the 4-bit ECC chips without as much as putting a
>> warning message in the Linux kernel, but that's a separate topic...).
>>
>> Anyhow, we modified the AT91bootstrap loader to convert the entire NAND
>> from 1-bit sw-ecc to on-die 4-bit ECC.  I'd be happy to send a patch to
>> this mailing list if you think this would be useful.
> 
> Yes, this would be great! The patch is certainly also helpful for the
> next with the same problem ...
> 
>>
>> UBI fortunately does not use OOB at all, so there should be no issue from
>> there.
> I have read this too. I'm happy that I have not to deal with YAFFS2.
> 
>>
>> The subpage size has to match though.  In our case, UBI was built for 512B
>> subpage, but when turning on the on-die ECC support, it'd by default only
>> support 2048B pages.  The patch I sent earlier to this mailing list enables
>> 512B subpage writes with on-die ECC.
> 
> That could be my problem. I have not considered this point. Thanks!
> 
> Best regards,
> Steffen
> 
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
> 

-- 
-------------------------------------
Angus Clark
ST Microelectronics (R&D) Ltd.
1000 Aztec West, Bristol, BS32 4SQ
email: angus.clark@st.com
tel: +44 (0) 1454 462389
st-tina: 065 2389
-------------------------------------

      reply	other threads:[~2013-04-12 10:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-11 16:29 Changing the ECC method for a running system (UBI seems to use OOB?) Steffen Kühn
2013-04-11 16:34 ` David Mosberger-Tang
2013-04-12  9:12   ` Steffen Kühn
2013-04-12 10:37     ` Angus CLARK [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=5167E3EF.9020505@st.com \
    --to=angus.clark@st.com \
    --cc=linux-mtd@lists.infradead.org \
    /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).