> -----Original Message----- > From: Vimal Singh [mailto:vimal.newwork@xxxxxxxxx] > Sent: Friday, January 21, 2011 1:08 PM > To: Ghorai, Sukumar > Cc: linux-omap@xxxxxxxxxxxxxxx; linux-mtd@xxxxxxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; Kamat, Nishant > Subject: Re: [PATCH] omap3: nand: bch ecc support added > > On Fri, Jan 21, 2011 at 9:37 AM, Ghorai, Sukumar <s-ghorai@xxxxxx> wrote: > > > >> -----Original Message----- > >> From: Vimal Singh [mailto:vimal.newwork@xxxxxxxxx] > >> Sent: Thursday, January 20, 2011 8:47 PM > >> To: Ghorai, Sukumar > >> Cc: linux-omap@xxxxxxxxxxxxxxx; linux-mtd@xxxxxxxxxxxxxxxxxxx; linux- > arm- > >> kernel@xxxxxxxxxxxxxxxxxxx; Kamat, Nishant > >> Subject: Re: [PATCH] omap3: nand: bch ecc support added > >> > >> Hi Ghorai, > >> > >> I am the initial author of this patch. And I guess this patch still > >> uses most of my work. > > [Ghorai] I know you are working on it, > > > > But I think your code was in a shape for - > > 1. not for up-streamble shape > That's because I never posted it for up-stream. > > > 2. non functional, > No, it was a working (at least for 8-bit correction) patch on that > kernel version (I do not remember the correct version now, may be > .29-.30) [Ghorai] can you send me the version? > It did not work for 4-bit, because of known reason. And I guess while > posting this patch you should also have mentioned/documented about > that. No? > > > So can you check this version matching with your original code? > > In that case I can add your Signed-off. > Yes, you should. I can see this version has got many changes, but > original idea is same. [Ghorai] there is no special idea needed to correct the ecc. Only correct algorithm is nedded. >[Ghorai]And not all code is changed. Does it? [Ghorai] can you send me your latest version? > > Regards, > Vimal -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html