public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] OMAP3 NAND ECC bug report
Date: Thu, 27 Oct 2011 14:02:56 -0500	[thread overview]
Message-ID: <4EA9AAE0.40100@freescale.com> (raw)
In-Reply-To: <CAB6Tvp35ndGndGOR_E0HOBrpacqnqssuwYOF5NH9D_j2x65aCA@mail.gmail.com>

On 10/27/2011 10:22 AM, Ivan Nardi wrote:
> On 20 October 2011 11:06, Arno Steffen <arno.steffen@googlemail.com> wrote:
>>
>> I modified single bits by adding a patch (thanks Scott Wood) to uboot,
>> that allows write in raw mode to flash. Single bit error can corrected
>> with HW-ECC, not with SW-ECC.
> 
> Hi all,
> 
> I would be very interested in using that patch (it is very useful for
> testing ecc algorithms)
> In u-boot the command "nand biterr" isn't implemented yet and some
> code found in the mailing list is pretty old and never merged (why?)
> into mainline (http://www.mail-archive.com/u-boot at lists.denx.de/msg22963.html)
> 
> Could you share it?
> Thanks a lot

raw read/write was recently added to mainline U-Boot:

commit fb3659ac422801ea18f36ef62926df70beeada4c
Author: Marek Vasut <marek.vasut@gmail.com>
Date:   Fri Sep 23 15:43:10 2011 +0200

    NAND: Add nand read.raw and write.raw commands

    These commands should work around various "hardware" ECC and BCH
methods.

    Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
    Cc: Scott Wood <scottwood@freescale.com>
    Cc: Stefano Babic <sbabic@denx.de>
    Cc: Wolfgang Denk <wd@denx.de>
    Cc: Detlev Zundel <dzu@denx.de>
    [scottwood at freescale.com: s/write the page/access the page/]
    Signed-off-by: Scott Wood <scottwood@freescale.com>

As for why the biterr patch wasn't merged, it's because nobody ever
resubmitted with a version that fixed the issues raised in review and
applies to the current tree.

-Scott

      reply	other threads:[~2011-10-27 19:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-20  9:06 [U-Boot] OMAP3 NAND ECC bug report Arno Steffen
2011-10-21  9:28 ` Arno Steffen
2011-10-25  7:41   ` Arno Steffen
2011-10-26 20:26     ` Scott Wood
2011-10-28  6:10       ` Arno Steffen
2011-10-28 16:06         ` Scott Wood
2011-10-31  9:36           ` Arno Steffen
2011-10-27 15:22 ` Ivan Nardi
2011-10-27 19:02   ` 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=4EA9AAE0.40100@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox