Messages in this thread | | | Date | Tue, 1 Dec 2015 21:41:33 +0530 | Subject | Re: [PATCH] mtd: Expand the ecc placement locations to 1216 | From | punnaiah choudary kalluri <> |
| |
Hi Boris,
On Tue, Dec 1, 2015 at 4:45 PM, Boris Brezillon <boris.brezillon@free-electrons.com> wrote: > Brian, Punnaiah, > > On Fri, 20 Nov 2015 11:33:42 -0800 > Brian Norris <computersforpeace@gmail.com> wrote: > >> On Thu, Nov 05, 2015 at 10:28:55PM +0530, Punnaiah Choudary Kalluri wrote: >> > Device like MT29F32G08ABCDBJ4 have a writesize/oobsize of 16K/1216 Bytes. >> > So, increasing the maximum ecc placement locations to 1216 >> >> I'd really prefer not increasing the size of the internal arrays any >> more. The structures should be rewritten to be dynamic. > > I already started this migration (you can find the code here [1]). I > was planning to rework this series to make the eccpos definition mimic > what's done for oobfree (as suggested by Brian) before sending it, but > I'm not sure when I'll be able to do that, so if you really need that > quickly I suggest you submit those patches instead of reimplementing > it yourself. >
Sure. let me check your implementation and then send these patches for review with your sign off.
Thanks, Punnaiah
> Best Regards, > > Boris > > [1]https://github.com/bbrezillon/linux-sunxi/commits/nand/ecclayout > > -- > Boris Brezillon, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com
| |