Re: [PATCH] mtd: Expand the ecc placement locations to 1216

From	punnaiah choudary kalluri <punnaia@xilinx.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] mtd: Expand the ecc placement locations to 1216
Date	2015-12-01 17:20 +0100
Message-ID	<qAVOO-3v2-19@gated-at.bofh.it> (permalink)
References	<qwZHk-363-27@gated-at.bofh.it> <qwZHk-363-25@gated-at.bofh.it> <qAR8u-sX-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] mtd: Expand the ecc placement locations to 1216 Boris Brezillon <boris.brezillon@free-electrons.com> - 2015-12-01 12:20 +0100
  Re: [PATCH] mtd: Expand the ecc placement locations to 1216 punnaiah choudary kalluri <punnaia@xilinx.com> - 2015-12-01 17:20 +0100

csiph-web