lkml.org 
[lkml]   [2012]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v2 0/3] mtd: nand: OMAP: ELM error correction support for BCH ecc
Date
On Thu, Nov 22, 2012 at 20:06:41, Philip, Avinash wrote:
> On Thu, Nov 22, 2012 at 16:13:52, Artem Bityutskiy wrote:
> > On Wed, 2012-11-21 at 07:01 +0000, Philip, Avinash wrote:
> > > > I am not sure how this dependency has to be handled for this series,
> > > > let me know whether you still want it to be made over l2-mtd?
> > >
> > > Artem,
> > >
> > > Is it possible for you to give ack for these patches so that these patches
> > > can go in Tony's tree where Omap-gpmc changes are present?
> >
> > I would prefer if people like Ivan could take a look at this first.
>
> Ok.

Ivan/Artem,

Any comments in this patch series?
I hope ELM support can get in 3.8.

>
> >
> > Also, I am not sure this is a good idea. Or at least we should agree on
> > some common strategy for bit-flips in the erased areas:
> >
> > "+ * 1. If page is erased, check with standard ecc vector (ecc vector
> > + * for erased page to find any bit flip). If check fails, bit flip
> > + * is present in erased page. Count the bit flips in erased page and
> > + * if it falls under correctable level, report page with 0xFF and
> > + * update the correctable bit information."
> >
>
> Idea here is to make faster scanning of erased page without bit flips.
> For omap nand driver ecc reported by hardware is non-zero and non 0xff.
> So comparing with the standard vector for erased page and skipping error
> correction for erased page without bit flips.
>
> Strategy for bit flips in erased page bit flips, can be
> 1. Don't make as erased page and mark it as bad.
> 2. Report the erased page with correctable bit flip if it falls under
> correctable level. Report the page with erased page with correctable
> errors.
> Is the concern is here with erased page with correctable error?
> I think as error is under correctable level, we still can use the page.
> May be we can think of limiting the check to half of correctable level?
>
> I would go for option 2, see discussion [1].
>
> Option 2 adopted in this patch series.
>
> > Basically, you are working-around JFFS2 limitations.
> >
> > Do you suggest we do this in all the drivers?
>
> I am not sure how the situation handled in other drivers.
> This depends on the platforms. This method can be adopted for
> platforms where ecc reported non-zero & non 0xff with erased page.

Any comments?

Thanks
Avinash

>
> >
> > If we want to do this, may be we better do this in higher level, common
> > to all drivers?
> >
>
> I doubt how we handle in higher level with existing MTD setup.
> Issues I am seeing on implementing at higher layer is
> 1. Calculation of standard ecc vector for erased page.
> 2. Skipping ecc error correction, as currently error correction in
> .read_page()
> Can be handled by adding common .read_page() method on certain flag,
> populated for platform specific.
>
> 1. http://lists.infradead.org/pipermail/linux-mtd/2011-April/034604.html
>
> Thanks
> Avinash
>
> > --
> > Best Regards,
> > Artem Bityutskiy
> >
>
>

\
 
 \ /
  Last update: 2012-11-27 10:41    [W:0.077 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site