RE: [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN

["Joakim Tjernlund" <Joakim.Tjernlund@transmode.se>]

> -----Original Message-----
> From: Nicolas Pitre [mailto:nico@cam.org]
> Sent: den 10 april 2008 01:02
> To: Joakim Tjernlund
> Cc: 'Alexey Korolev'; 'Jörn Engel'; haokexin@gmail.com; dwmw2@infradead.org; akpm@linux-
> foundation.org; linux-mtd@lists.infradead.org
> Subject: RE: [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN
> 
> On Thu, 10 Apr 2008, Joakim Tjernlund wrote:
> 
> > > -----Original Message-----
> > > From: linux-mtd-bounces@lists.infradead.org [mailto:linux-mtd-bounces@lists.infradead.org] On
> Behalf
> > > Of Alexey Korolev
> > > Sent: den 4 april 2008 14:22
> > > To: Jörn Engel
> > > Cc: nico@cam.org; haokexin@gmail.com; dwmw2@infradead.org; akpm@linux-foundation.org; linux-
> > > mtd@lists.infradead.org
> > > Subject: Re: [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN
> > >
> > > Hi
> > >
> > >
> > >  CFI driver in 2.6.24 kernel is broken. Not so intensive read/write operations cause incomplete
> writes
> > > which lead to kernel panics in JFFS2.
> > >  We investigated the issue - it is caused by bug in FL_SHUTDOWN parsing code. Sometimes chip
> returns -
> > > EIO as if it is in FL_SHUTDOWN state when it should wait in FL_PONT (error in order of
> conditions).
> > >
> > > The following patch fixes the bug in state parsing code of CFI.
> > > Also I've added comments to notify developers if they want to add new case in future.
> > > Please include it.
> > >
> > > Signed-off-by: Alexey Korolev <akorolev@infradead.org>
> > > Reviewed-By: Joern Engel <joern@logfs.org>
> >
> > Acked-By: Joakim Tjernlund <Joakim.Tjenrlund@transmode.se>
> >
> > Guys, I think you should sent this directly to Linus as David seems
> > unavailable ATM. It should really go into 2.6.25
> 
> It is already in mainline.  See commit fb6d080c.

Oh, good. Never saw any mail about it on the list, sorry for the noise.

     Jocke