From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Thu, 27 Oct 2011 14:02:56 -0500
Subject: [U-Boot] OMAP3 NAND ECC bug report
In-Reply-To: <CAB6Tvp35ndGndGOR_E0HOBrpacqnqssuwYOF5NH9D_j2x65aCA@mail.gmail.com>
References: <CAACX+R2hzKo898GqjFHU6rxw_-11htVm+qyMH_9wXLE1nbBq8w@mail.gmail.com>
	<CAB6Tvp35ndGndGOR_E0HOBrpacqnqssuwYOF5NH9D_j2x65aCA@mail.gmail.com>
Message-ID: <4EA9AAE0.40100@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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